-
Notifications
You must be signed in to change notification settings - Fork 245
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
btrfs-progs: README: PR workflow, minor fixes
Mention the basic pull request workflow, the rest are style fixes. [skip ci] Signed-off-by: David Sterba <[email protected]>
- Loading branch information
Showing
1 changed file
with
17 additions
and
5 deletions.
There are no files selected for viewing
This file contains 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 |
---|---|---|
|
@@ -45,7 +45,7 @@ There are several ways, each has its own specifics and audience that can give | |
feedback or work on a fix. The following list is sorted in the order of | ||
preference: | ||
|
||
* [github issue tracker](https://github.com/kdave/btrfs-progs/issues) | ||
* [Github issue tracker](https://github.com/kdave/btrfs-progs/issues) | ||
* to the mailing list *[email protected]* -- (not required to | ||
subscribe), beware that the mail might get overlooked in other traffic | ||
* IRC (irc.libera.chat #btrfs) -- good for discussions eg. if a bug is already | ||
|
@@ -57,15 +57,15 @@ Development | |
----------- | ||
|
||
The development takes place in the mailing list (*[email protected]*) | ||
or at github (issues, pull requests). Changes should be split to logical parts | ||
or at Github (issues, pull requests). Changes should be split to logical parts | ||
if possible, documentation may be included in the same patch as to code or | ||
separately. | ||
|
||
The development model of btrfs-progs shares a lot with the kernel model. The | ||
|
||
* **one logical change per patch**: eg. not mixing bugfixes, cleanups, features | ||
* **one logical change per patch**: e.g. not mixing bugfixes, cleanups, features | ||
etc., sometimes it's not clear and will be usually pointed out during reviews | ||
* proper **subject line**: eg. prefix with _btrfs-progs: subpart, ..._ , | ||
* proper **subject line**: e.g. prefix with _btrfs-progs: subpart, ..._ , | ||
descriptive yet not too long, see `git log --oneline` for some inspiration | ||
* proper **changelog**: the changelogs are often missing or lacking explanation _why_ | ||
the change was made, or _how_ is something broken, _what_ are user-visible | ||
|
@@ -80,6 +80,18 @@ The development model of btrfs-progs shares a lot with the kernel model. The | |
substitute in order to allow contributions without much bothering with | ||
formalities | ||
|
||
### Pull requests | ||
|
||
The pull requests on Github may be used for code or documentation | ||
contributions. There are basic build checks enabled in the Github actions CI | ||
(first time contributors' pull requests may need an approval). The status can | ||
be checked at the [workflow page](https://github.com/kdave/btrfs-progs/actions/workflows/pull-request.yml). | ||
|
||
* open a PR against branches *devel* or *master* | ||
* push update to the same branch if you need to | ||
* close the PR in case it's wrong, a mistake or needs rework | ||
* if you're sure the changes don't need a CI build verification, please add `[skip ci]` to the changelog | ||
|
||
Source code coding style and preferences follow the | ||
[kernel coding style](https://www.kernel.org/doc/html/latest/process/coding-style.html). | ||
You can find the editor settings in `.editorconfig` and use the | ||
|
@@ -98,7 +110,7 @@ Documentation updates | |
Documentation fixes or updates do not need much explanation so sticking to the | ||
code rules in the previous section is not necessary. GitHub pull requests are | ||
OK, patches could be sent to me directly and not required to be also in the | ||
mailinglist. Pointing out typos via IRC also works, although might get | ||
mailing list. Pointing out typos via IRC also works, although might get | ||
accidentally lost in the noise. | ||
|
||
Documentation sources are written in | ||
|