-
-
Notifications
You must be signed in to change notification settings - Fork 690
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
Modified docs as per suggestions and cleared the mess #40
Conversation
Added paragraph
Are you creating a new PR every time you are making a modification? Please don't do that - just push new commits to the existing PR. You have 4 open pull requests now - #36, #38, #39, #40, and there seems to be dependencies and duplication between them. This makes it too hard for us to approve your pull requests.
Have a read through the contribution guidelines again please. |
@aslakhellesoy : yeah i messed up and confused every i created a pull request. My sincere apologies for creating a mess. |
No worries @gmk1217 - we're very grateful for your help! Maybe it's best if you close all your existing pull requests and submit new ones, where each PR is for one document? And none of the PRs depend on other PRs? I'm still concerned about copying content from other sites. All our content should be original. |
ok will close duplicate PRs . |
Thanks for working with us on this @gmk1217. In general, small PRs are better - when there's less to consider and discuss, they'll flow through the review process faster. |
Agreed. Will adopt and make thngs easy for you guys. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Modified documents as per suggested changes added cucumber.md and living documentation.md