-
-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
Update the CONTRIBUTING GUIDE #3327
Conversation
CONTRIBUTING.md
Outdated
@@ -14,214 +8,143 @@ Strapi is an open-source project administered by [the Strapi team](https://strap | |||
|
|||
Before contributing, ensure that your effort is aligned with the project's roadmap by talking to the maintainers, especially if you are going to spend a lot of time on it. Feel free to [join us on Slack](http://slack.strapi.io) if you are interested in helping us or [drop us an email](mailto:hi@strapi.io) if you are interested in working with us. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you are interesting in helping us or working with us
CONTRIBUTING.md
Outdated
|
||
If you send a pull request, please do it against the `master` branch. We are developing upcoming versions separately to ensure non-breaking changes from master to the latest stable major version. | ||
We are using [GitHub Issues](https://github.com/strapi/strapi/issues) to manage our public bugs. We keep a close eye on this so before filling a new task try to make sure the problem doesn't already exist. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so before filling a new issue, try to make sur it does not already exist
CONTRIBUTING.md
Outdated
|
||
[Go to the repository](https://github.com/strapi/strapi) and fork it to your own GitHub account. | ||
**Before submitting your pull request** make sure the following requirements are done: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the following requirements are fullfilled
CONTRIBUTING.md
Outdated
|
||
#### 3. ⏳ Installation | ||
- You have [Node](https://nodejs.org/en/) at v8.0.0+ and [Yarn](https://yarnpkg.com/en/) at v1.2.0+. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Node v10 only ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@renatodex Yarn is now required to contribute as we are exploiting the yarn workspaces feature to manage the monorepo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please comments. A few typos and a few recommendations/questions.
Description:
This PR updates the CONTRIBUTING GUIDE to reflect the new development workflow.
My PR is a:
Main update on the:
Manual testing done on the following databases: