#33043 closed task (blessed) (fixed)
Add contributing.md to git mirror
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 5.7 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Build/Test Tools | Keywords: | has-patch |
Focuses: | Cc: |
Description
As is apparent from the 100+ pull requests that have been submitted, devs dont read the description shown on github and create pull requests anyway. One way to limit this noise, and guide devs to the right place is to make use of Contributing guidelines so that when they try and create a pull request they see a call to action to review the guidelines.
The attached diff would make this appear when users try to create a pull request:
And if they click the guidelines link they'd see this:
Attachments (1)
Change History (20)
#2
follow-up:
↓ 4
@
9 years ago
The new version of github accepts [ISSUE_TEMPLATE.md](https://help.github.com/articles/creating-an-issue-template-for-your-repository/) and [PULL_REQUEST_TEMPLATE.md](https://help.github.com/articles/creating-a-pull-request-template-for-your-repository/)
Those files appear to anyone who is trying to add a PULL REQUEST and also for who wants to OPEN AN ISSUE.
With that said, it'll avoid anyone trying to contribute via github.
#3
@
9 years ago
- Keywords needs-patch added
- Milestone changed from Awaiting Review to Future Release
Yup, let's do this.
- "read only" should be "read-only"
- "WordPress", not "wordpress"
- Use https URL
Let's also link to https://make.wordpress.org/core/handbook/, our official contributor documentation.
#4
in reply to:
↑ 2
@
9 years ago
Replying to leocaseiro:
The new version of github accepts [ISSUE_TEMPLATE.md](https://help.github.com/articles/creating-an-issue-template-for-your-repository/) and [PULL_REQUEST_TEMPLATE.md](https://help.github.com/articles/creating-a-pull-request-template-for-your-repository/)
Those files appear to anyone who is trying to add a PULL REQUEST and also for who wants to OPEN AN ISSUE.
Issues are closed/disabled so no need for an issues template.
Some example templates, or for a little more inspiration the Choose Your Own Adventure template generator.
#6
@
4 years ago
- Keywords needs-refresh added; needs-patch removed
- Milestone changed from Future Release to 5.7
- Owner set to desrosj
- Status changed from new to assigned
Going to assign this to myself and get a CONTRIBUTING.md file committed. However, I am not sure that the https://github.com/wordpress/wordpress mirror will ever contain this file as that mirrors the build repository.
However, we can add one to the https://github.com/wordpress/wordpress-develop repository and update the language of the message closing the PR. Maybe including a link to https://make.wordpress.org/core/handbook/contribute/git/github-pull-requests-for-code-review/.
Moving to 5.7 as it's not likely I'll get to this prior to 5.6's release in a few weeks.
This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.
4 years ago
This ticket was mentioned in Slack in #core by lukecarbis. View the logs.
4 years ago
#9
@
4 years ago
Just noting that we do allow PRs against that repo now, so long as they're linked to a Trac ticket.
Nonetheless we should add a README.md which informs folks of this :)
This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.
4 years ago
#11
@
4 years ago
- Type changed from enhancement to task (blessed)
Moving this ticket from enhancement
to task (blessed)
after discussion in slack. Why? It does not impact the packaged codebase. As a task, work can continue throughout the entire cycle.
This ticket was mentioned in PR #1039 on WordPress/wordpress-develop by kraftbj.
4 years ago
#12
- Keywords has-patch added; needs-refresh removed
Adds a CONTRIBUTING.md to aid in helping direct folks to more information about contributing to WordPress.
This ticket was mentioned in Slack in #core by desrosj. View the logs.
4 years ago
4 years ago
#14
@desrosj
We have a Welcome message that gets posted when someone opens their first PR. It seems most of the information in that post is reflected here. But I'm wondering how these two should work together going forward.
I think they serve similar complementary purposes. The CONTRIBUTING.md
suggested here would allow for a notice to show before submitting the PR (as seen here) while the action pushes a comment post-submission.
For the initial commit, I think having them as separate things is okay (after adding your suggested amendment), but we can look at having the GitHub action pull from the CONTRIBUTING.md
and posting it as the comment body after the welcome to DRY it out.
4 years ago
#15
@desrosj
We have a Welcome message that gets posted when someone opens their first PR. It seems most of the information in that post is reflected here. But I'm wondering how these two should work together going forward.
I think they serve similar complementary purposes. The CONTRIBUTING.md
suggested here would allow for a notice to show before submitting the PR (as seen here) while the action pushes a comment post-submission.
For the initial commit, I think having them as separate things is okay (after adding your suggested amendment), but we can look at having the GitHub action pull from the CONTRIBUTING.md
and posting it as the comment body after the welcome to DRY it out.
4 years ago
#17
Merged into Core in https://core.trac.wordpress.org/changeset/50417.
Thanks AD7six,
Makes sense, it also doesn't mean people will read this one either, but worth a shot ;)