-
Notifications
You must be signed in to change notification settings - Fork 229
README: Replace local contrib docs with a CONTRIBUTING.md link #9
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
Merged
+8
−107
Merged
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -24,116 +24,17 @@ Following sections give context for aspects of this specification: | |
|
|
||
| ## Contributing | ||
|
|
||
| Development happens on GitHub for the spec. Issues are used for bugs and actionable items and longer discussions can happen on the [mailing list](https://github.com/opencontainers/distribution-spec#mailing-list). | ||
| See [our contribution documentation](CONTRIBUTING.md). | ||
|
|
||
| The specification and code is licensed under the Apache 2.0 license found in the [LICENSE](https://github.com/opencontainers/distribution-spec/blob/master/LICENSE) file. | ||
| Development happens on GitHub. | ||
| Responsible disclosure for security issues is discussed [here](CONTRIBUTING.md#security-issues). | ||
| [Issues][issues] are used for non-security bugs and actionable items; longer discussions can happen on the [mailing list](#mailing-list). | ||
|
|
||
| ### Discuss your design | ||
| ### Mailing list | ||
|
|
||
| The project welcomes submissions, but please let everyone know what you are working on. | ||
|
|
||
| Before undertaking a nontrivial change to this specification, send mail to the [mailing list](#mailing-list) to discuss what you plan to do. | ||
| This gives everyone a chance to validate the design, helps prevent duplication of effort, and ensures that the idea fits. | ||
| It also guarantees that the design is sound before code is written; a GitHub pull-request is not the place for high-level discussions. | ||
|
|
||
| Typos and grammatical errors can go straight to a pull-request. | ||
| When in doubt, start on the [mailing-list](#mailing-list). | ||
|
|
||
| ### Meetings | ||
|
|
||
| The contributors and maintainers of all OCI projects have monthly meetings at 2:00 PM (USA Pacific) on the first Wednesday of every month. | ||
| There is an [iCalendar][rfc5545] format for the meetings [here](https://github.com/opencontainers/runtime-spec/blob/master/meeting.ics). | ||
| Everyone is welcome to participate via [UberConference web][uberconference] or audio-only: +1 415 968 0849 (no PIN needed). | ||
| An initial agenda will be posted to the [mailing list](#mailing-list) in the week before each meeting, and everyone is welcome to propose additional topics or suggest other agenda alterations there. | ||
| Minutes are posted to the [mailing list](#mailing-list) and minutes from past calls are archived [here][minutes], with minutes from especially old meetings (September 2015 and earlier) archived [here][runtime-wiki]. | ||
|
|
||
| ### Mailing List | ||
|
|
||
| You can subscribe and join the mailing list on [Google Groups][dev-list]. | ||
|
|
||
| ### IRC | ||
|
|
||
| OCI discussion happens on #opencontainers on Freenode ([logs][irc-logs]). | ||
|
|
||
| ### Git commit | ||
|
|
||
| #### Sign your work | ||
|
|
||
| The sign-off is a simple line at the end of the explanation for the patch, which certifies that you wrote it or otherwise have the right to pass it on as an open-source patch. | ||
| The rules are pretty simple: if you can certify the below (from http://developercertificate.org): | ||
|
|
||
| ``` | ||
| Developer Certificate of Origin | ||
| Version 1.1 | ||
| Copyright (C) 2004, 2006 The Linux Foundation and its contributors. | ||
| 660 York Street, Suite 102, | ||
| San Francisco, CA 94110 USA | ||
| Everyone is permitted to copy and distribute verbatim copies of this | ||
| license document, but changing it is not allowed. | ||
| Developer's Certificate of Origin 1.1 | ||
| By making a contribution to this project, I certify that: | ||
| (a) The contribution was created in whole or in part by me and I | ||
| have the right to submit it under the open source license | ||
| indicated in the file; or | ||
| (b) The contribution is based upon previous work that, to the best | ||
| of my knowledge, is covered under an appropriate open source | ||
| license and I have the right under that license to submit that | ||
| work with modifications, whether created in whole or in part | ||
| by me, under the same open source license (unless I am | ||
| permitted to submit under a different license), as indicated | ||
| in the file; or | ||
| (c) The contribution was provided directly to me by some other | ||
| person who certified (a), (b) or (c) and I have not modified | ||
| it. | ||
| (d) I understand and agree that this project and the contribution | ||
| are public and that a record of the contribution (including all | ||
| personal information I submit with it, including my sign-off) is | ||
| maintained indefinitely and may be redistributed consistent with | ||
| this project or the open source license(s) involved. | ||
| ``` | ||
|
|
||
| then you just add a line to every git commit message: | ||
|
|
||
| Signed-off-by: Joe Smith <[email protected]> | ||
|
|
||
| using your real name (sorry, no pseudonyms or anonymous contributions.) | ||
|
|
||
| You can add the sign off when creating the git commit via `git commit -s`. | ||
|
|
||
| #### Commit Style | ||
|
|
||
| Simple house-keeping for clean git history. | ||
| Read more on [How to Write a Git Commit Message][how-to-git-commit] or the Discussion section of [git-commit(1)][git-commit.1]. | ||
|
|
||
| 1. Separate the subject from body with a blank line | ||
| 2. Limit the subject line to 50 characters | ||
| 3. Capitalize the subject line | ||
| 4. Do not end the subject line with a period | ||
| 5. Use the imperative mood in the subject line | ||
| 6. Wrap the body at 72 characters | ||
| 7. Use the body to explain what and why vs. how | ||
| * If there was important/useful/essential conversation or information, copy or include a reference | ||
| 8. When possible, one keyword to scope the change in the subject (i.e. "README: ...", "runtime: ...") | ||
| You can subscribe and browse the mailing list on [Google Groups][mailing-list]. | ||
|
|
||
| [charter]: https://www.opencontainers.org/about/governance | ||
| [code-of-conduct]: https://github.com/opencontainers/tob/blob/master/code-of-conduct.md | ||
| [dev-list]: https://groups.google.com/a/opencontainers.org/forum/#!forum/dev | ||
| [how-to-git-commit]: http://chris.beams.io/posts/git-commit | ||
| [irc-logs]: http://ircbot.wl.linuxfoundation.org/eavesdrop/%23opencontainers/ | ||
| [iso-week]: https://en.wikipedia.org/wiki/ISO_week_date#Calculating_the_week_number_of_a_given_date | ||
| [minutes]: http://ircbot.wl.linuxfoundation.org/meetings/opencontainers/ | ||
| [oci]: https://www.opencontainers.org | ||
| [rfc5545]: https://tools.ietf.org/html/rfc5545 | ||
| [runtime-wiki]: https://github.com/opencontainers/runtime-spec/wiki | ||
| [uberconference]: https://www.uberconference.com/opencontainers | ||
|
|
||
| [git-commit.1]: http://git-scm.com/docs/git-commit | ||
| [issues]: https://github.com/opencontainers/distribution-spec/issues | ||
| [mailing-list]: https://groups.google.com/a/opencontainers.org/forum/#!forum/dev | ||
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.
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 at least provide a link to the issue tracker and mailing list so users don't need to wade through several docs before finding basic information. The longer form stuff is definitely part of contributing.
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.
Done with 724ab29 → 2bbc5c1, using text copy/pasted from
CONTRIBUTING.md. If you have specific wording in mind, give me something to copy/paste ;).If you're concerned about skimmers reading some of the contribution information (e.g. the issues link in the README with 2bbc5c1) and submitting security issues over those public channels, you may want to add a reference to the security section as well. For a paranoid approach to this “skimmed and missed the security section” issue, see opencontainers/project-template#57.
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.
Please do add this.
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.
Done with 2bbc5c1 → fa939cc.