ICU-22309 update CONTRIBUTING.md for new process #2417
Conversation
srl295
force-pushed
the
srl295/icu-22309/contributing-deux
branch
from
5754ded
to
f198d16
Compare
|
Hooray! The files in the branch are the same across the force-push. 😃 ~ Your Friendly Jira-GitHub PR Checker Bot |
srl295
force-pushed
the
srl295/icu-22309/contributing-deux
branch
from
f198d16
to
d40e714
Compare
|
Notice: the branch changed across the force-push!
~ Your Friendly Jira-GitHub PR Checker Bot |
|
@markusicu ping ? required for new CLA process |
|
|
||
| ## Process | ||
|
|
||
| See also [git for ICU Developers][git4icu]. | ||
|
|
||
| If you want to join the team, then please contact us. Once everything is agreed, the ICU team adds you to the GitHub project and the Jira issue tracker. | ||
| If you want to join the team, then please [contact us][contacts]. Once everything is agreed, the ICU team adds you to the [GitHub project][unicode-org/icu] and the [Jira issue tracker][bugs]. |
There was a problem hiding this comment.
It looks like casual patches (without joining the team) got lost in the shuffle. I suggest we bring it back roughly here:
| If you want to join the team, then please [contact us][contacts]. Once everything is agreed, the ICU team adds you to the [GitHub project][unicode-org/icu] and the [Jira issue tracker][bugs]. | |
| For patches, please submit both a Jira ticket and a GitHub pull request. See [Submitting ICU Bugs and Feature Requests][bugs]. | |
| If you want to join the team, then please [contact us][contacts]. Once everything is agreed, the ICU team adds you to the [GitHub project][unicode-org/icu] and the [Jira issue tracker][bugs]. |
There was a problem hiding this comment.
There was a problem hiding this comment.
@markusicu could we consider this in a separate PR? So that the wording for the already-installed CLA is in place first?
There was a problem hiding this comment.
It seems like it could have been quick to amend the commit... but if you can remember to make another PR then that's ok.
|
|
||
| ## Process | ||
|
|
||
| See also [git for ICU Developers][git4icu]. | ||
|
|
||
| If you want to join the team, then please contact us. Once everything is agreed, the ICU team adds you to the GitHub project and the Jira issue tracker. | ||
| If you want to join the team, then please [contact us][contacts]. Once everything is agreed, the ICU team adds you to the [GitHub project][unicode-org/icu] and the [Jira issue tracker][bugs]. |
There was a problem hiding this comment.
It seems like it could have been quick to amend the commit... but if you can remember to make another PR then that's ok.
Checklist
#2402 was the prior update to CONTRIBUTING.md
#2415 also see
CLA section from https://github.com/unicode-org/.github/blob/main/.github/CONTRIBUTING.md
fyi @annebright