Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Contributing.md Enhancement #1683
base: gh-pages
Are you sure you want to change the base?
Contributing.md Enhancement #1683
Changes from 12 commits
1d1346d
88c992f
518a22f
9f2d071
ad7cd69
3e3116a
823e960
2addced
2371fa1
b17d5d1
e07a240
720563a
9d53571
34977ba
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
is approval necessary? who is delegated the authority to make these approvals?
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.
This should not be in here at all imo. The process as with any other repo in this org, is "if it is complicated and/or you dont want to waste your time on a PR that my not land, ask if it is a good idea, otherwise open a 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.
I am going to copy over a quote from the discussion on this from the slack.
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.
I tried to remove any offensive language around this, and can change the wording again, but this idea was literally the point of redoing this document. Otherwise we might be good to just leave things as they are currently.
This was mentioned before I started this #1506 (comment).
@bjohansebas Did we not want this step in our process? In my observations, we have been following this process for the past few months at least.
Approval is informal and can be done by any member.
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.
Here is another instance where we used this. #1568 (comment) which resulted in #1590
Can we reword this idea somehow to make it more acceptable? Something like:
If you have any questions or concerns about the utility of your submission, consider opening an issue first to discuss with the team
, etcThere 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.
How about something like this:
"You're strongly encouraged to open a discussion issue before starting work on a PR, particularly for significant changes that have wide impact. Small minor improvements or corrections don't need an issue, but for more pervasive contributions, an issue helps to clarify and focus the work and ensure it aligns with the overall project priorities."
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.
I've made some changes to the wording in those sections and removed the TL;DR part.
I tried to soften it, make it seem optional, remove all references to "approval", and and say it's for large changes only.