Closed jafeltra closed 4 months ago
Thanks, Julia. This looks great -- and I think that potential contributors will really appreciate it! I also like the Code of Conduct badge and tab.
As for the location of the files, I prefer having them in the root directory of the project so that others easily notice them. Up until today, I wouldn't have thought of looking in .github
for files like this.
@cmoesel and @mint-thompson: I made some small tweaks based on feedback from @KaelynJefferson. Let me know what you think.
Also, I just wanted to call out two other thoughts to make sure we had considered:
@jafeltra HL7 has reviewed it and noted that we should use the HL7 Code of Conduct instead of the one we're currently pointing to. Please:
To ensure a welcoming environment, we follow the HL7 Code of Conduct and expect contributors to do the same. For your convenience, a copy of the HL7 Code of Conduct, as of June 2024, is included below. Please visit the official HL7 Code of Conduct for the most recent version.
(HL7 Code of Conduct text goes here...)
Just documenting what we discussed in our 1:1: It's sufficient to have the CODE-OF-CONDUCT file link to the HL7 Code of Conduct without copying all its text.
@cmoesel - I just updated this PR with the feedback from HL7 and the approach you noted above.
This PR adds a Contribution Policy for SUSHI. This includes a
CONTRIBUTING.md
file and aCODE-OF-CONDUCT.md
file.The Code of Conduct comes from https://www.contributor-covenant.org/. They mentioned there is a badge, so I added it, but it isn't required. If you look at the branch in the main code viewer, you can see the badge in the README and see the top-level link that GitHub adds for the Code of Conduct at the top of the README viewer.
One other note: these files are both in the root of the repository, but I learned that you can alternatively add them to the
.github
directory. That's actually where the github docs repo has these files. On one hand, it keeps these types of files all in one place. On the other hand, they're a little less discoverable. Let me know if anyone has an opinion on the placement.