X-Git-Url: http://git.openwrt.org/?a=blobdiff_plain;f=CONTRIBUTING.md;h=4bf5626c4f5f5dc7db78125547bb796c25941f7b;hb=fa69fdc2b081abb57fc05c8d16356b16bdfdec5c;hp=993915b93f85c636851dc601fc262bd43d750492;hpb=70156c9682c01cbfae23a37a2c835188853b010d;p=feed%2Fpackages.git diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 993915b93f..4bf5626c4f 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -27,9 +27,9 @@ guidelines: - Convenience macros for popular mirrors are defined. Using these macros will make your package downloads more robust by mapping to a list of possible source mirrors for archive availability. - - @SF - Sourceforge (downloads.sourceforge.net) with 5 retries due to + - @SF - SourceForge (downloads.sourceforge.net) with 5 retries due to re-directs - - @GITHUB - Github (raw.githubusercontent.com) with 5 retries due to + - @GITHUB - GitHub (raw.githubusercontent.com) with 5 retries due to re-directs - @GNU - 8 regional servers - @GNOME - 8 regional servers @@ -70,18 +70,22 @@ guidelines: ### Commits in your pull-requests should -- Have a useful description prefixed with the package name (E.g.: "foopkg: Add - libzot dependency") +- Have a useful commit subject prefixed with the package name (E.g.: "foopkg: + Add libzot dependency"). - Include Signed-off-by tag in the commit comments. See: [Sign your work](https://openwrt.org/submitting-patches#sign_your_work) +- Author and sign-off must match and be a real name or known identity and + a real email address. GitHub private email addresses will not be accepted. +- Follow all [Submission Guidelines](https://openwrt.org/submitting-patches#submission_guidelines) + requirements, including maximum characters per line. ## Advice on pull requests -Pull requests are the easiest way to contribute changes to git repos at Github. +Pull requests are the easiest way to contribute changes to git repos at GitHub. They are the preferred contribution method, as they offer a nice way for commenting and amending the proposed changes. -- You need a local "fork" of the Github repo. +- You need a local "fork" of the GitHub repo. - Use a "feature branch" for your changes. That separates the changes in the pull request from your other changes and makes it easy to edit/amend commits @@ -89,10 +93,10 @@ commenting and amending the proposed changes. - Update your local git fork to the tip (of the master, usually) - Create the feature branch with `git checkout -b feature_x` - Edit changes and commit them locally - - Push them to your Github fork by `git push -u origin feature_x`. That - creates the "feature_x" branch at your Github fork and sets it as the + - Push them to your GitHub fork by `git push -u origin feature_x`. That + creates the "feature_x" branch at your GitHub fork and sets it as the remote of this branch - - When you now visit Github, you should see a proposal to create a pull + - When you now visit GitHub, you should see a proposal to create a pull request - If you later need to add new commits to the pull request, you can simply @@ -109,7 +113,7 @@ commenting and amending the proposed changes. - If you added several new commits or made other changes that require cleaning up, you can use `git rebase -i HEAD~X` (X = number of commits to edit) to possibly squash some commits - - Push the changed commits to Github with `git push -f` to overwrite the + - Push the changed commits to GitHub with `git push -f` to overwrite the original commits in the "feature_x" branch with the new ones. The pull request gets automatically updated @@ -200,9 +204,9 @@ checking if `foobar --version` prints the correct version. `PKG_VERSION` is the OpenWrt version and therefore includes the `PKG_RELEASE`, which isn't usually part of the running programs version. -The following snippet show a script that tests different binaries, depending +The following snippet shows a script that tests different binaries depending on what IPK package was installed. The `gpsd` Makefile produces both a `gpsd` and -a `gpsd-clients` IPK package. +a `gpsd-clients` IPK packages. ```shell #!/bin/sh