-
Notifications
You must be signed in to change notification settings - Fork 12
[3.4] Improve project description #9
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
Conversation
Why could anyone want to create a new project based on this repository? |
To reproduce a bug and share it for investigation.
The purpose of the project is not to serve as a template. |
Is it possible to raise a pull request to this repository from project created using |
also not possible with |
If replacing the long way with the short one does not convince you (I repeat, the behavior is exactly the same), I can modify the PRs so that the long version remains as an option. I can also include a description of what is the expected use of the project. I just need you to tell me what changes you would like. |
Including a link to the contribution guides could also be helpful. |
Then these instructions aren't (and weren't) fit for purpose "to reproduce a bug and share it for investigation". I think that it would be the best to delete installation instructions and replace them with link to your contribution guide https://github.com/Codeception/module-symfony/blob/master/CONTRIBUTING.md |
well, at some point you had to install the project 😆 . OK, I'll make the changes at the end of the day. Thanks for the feedback |
@Naktibalda I made the changes. Is it better this way? |
I think you shouldn't remove the installation instructions, since they're referenced at https://github.com/Codeception/module-symfony/blob/master/CONTRIBUTING.md
|
What Naktibalda suggests is that they be added to the contribution guides. Since the exact command you must run if you want to contribute involves cloning your fork, and not cloning the original repository. |
@ThomasLandauer Can you please include the 3 commands that I remove here in the contribution guides?
It can be in the PR that you already have open. |
Just for the record i proposed this because installing phalcon/demo with composer is already possible and even the test project readme explains both |
No description provided.