-
Notifications
You must be signed in to change notification settings - Fork 67
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
docs: Add CONTRIBUTING.md
file
#452
Conversation
edd0e34
to
c1fc487
Compare
BTW I just noticed that when a
automatically generated with a link to that file + the issues tagged |
c1fc487
to
dc4a03a
Compare
FYI @yurug I also updated the wiki, https://github.com/ocaml-sf/learn-ocaml/wiki/Checklist-for-testing-and-merging-a-PR |
This patch, just required if PR <ocaml-sf#453> is merged before PR <ocaml-sf#452>, can be reverted anytime soon.
This patch, just required if PR <ocaml-sf#453> is merged before PR <ocaml-sf#452>, can be reverted anytime soon.
dc4a03a
to
1eeb0e0
Compare
f5ba0ba
to
ce9757c
Compare
ce9757c
to
ad479f4
Compare
LGTM! |
This commit refactors and extends the text from:
https://github.com/ocaml-sf/learn-ocaml/wiki/Checklist-for-testing-and-merging-a-PR
which mostly results from standard conventions + those that are necessary because of the
release-please
dependency.Proofreading is welcome!
(Cc @yurug @AltGr FYI)
Direct link to the rendered
.md
[updated]