Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhancement: Link to project documentation #652

Merged
merged 1 commit into from
May 7, 2018

Conversation

localheinz
Copy link
Contributor

@localheinz localheinz commented May 7, 2018

This PR

  • links to the project documentation in composer.json

Follows #639 (comment).

@jwage jwage added this to the 2.0 milestone May 7, 2018
@jwage jwage self-assigned this May 7, 2018
composer.json Outdated
],
"support": {
"issues": "https://github.com/doctrine/migrations/issues",
"source": "https://github.com/doctrine/migrations",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these 2 are not necessary, as they are already auto-detected by Composer AFAIK.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shall we remove the entire section in favor of #639 (comment), then?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If what @stof says above is true, then 👍

@localheinz localheinz force-pushed the feature/support branch 2 times, most recently from 19dc032 to 550b8fa Compare May 7, 2018 14:28
@localheinz localheinz changed the title Enhancement: Add support section to composer.json Enhancement: Link to project documentation May 7, 2018
@jwage jwage merged commit 0469ded into doctrine:master May 7, 2018
@localheinz localheinz deleted the feature/support branch May 7, 2018 21:55
@localheinz
Copy link
Contributor Author

Thank you, @jwage and @stof!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants