-
-
Notifications
You must be signed in to change notification settings - Fork 313
Update CONTRIBUTING.md to remove outdated information #1416
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
Summary: This file is a brand new version of the contributing file to remove outdate information. Related issue: #1414
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.
Made a few suggestions for change.
I think this would be easier to review by spreading the PR over multiple commits.
There are a lot of things that no longer apply, but we don't exactly have a replacememt process ready. Such as, we shouldn't mention drafts or milestones, because right now we aren't using them in any defined way.
This is a good start though for sure.
Pinging @benjagm (just a reminder). |
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.
Looks good to me. I can't approve the PR with my permissions.
@benjagm it's your PR. I think @Relequestual's comments are for you to address. 😉 |
Thanks Greg! I totally forgot about this one 🤪 |
Co-authored-by: Ben Hutton <[email protected]>
Co-authored-by: Ben Hutton <[email protected]>
Co-authored-by: Ben Hutton <[email protected]>
Adding suggestion made in the PR.
Thanks everyone for the reviews and comments. I tried to address all the changes with the last PR. |
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.
Sorry for the delay. Looks all good! Thanks.
Summary: This file is a brand new version of the contributing file to remove outdate information.
Resolves #1414