From c5293edbabc58ab58516a75e60bbf1b6beaf4dcc Mon Sep 17 00:00:00 2001 From: David Rheinsberg Date: Tue, 31 Mar 2020 11:13:04 +0200 Subject: [PATCH] docs: break lines in markdown Avoid overlong lines in markdown files. This does not affect the rendered output, but makes it a lot easier to navigate through the file from a text-editor when editing. Furthermore, it improves *Diffs* of our commits, and allows reading markdown without rendering it first. --- CONTRIBUTING.md | 41 +++++++++++++++++++++++++++++------------ 1 file changed, 29 insertions(+), 12 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index fc6d8863a..2155b68b2 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -1,10 +1,14 @@ # Contributing to osbuild-composer -First of all, thank you for taking the time to contribute to osbuild-composer. In this document you will find information that can help you with your contribution. +First of all, thank you for taking the time to contribute to osbuild-composer. +In this document you will find information that can help you with your +contribution. ### Running from sources -We recommend using the latest stable Fedora for development but latest minor release of RHEL/CentOS 8 should work just fine as well. To run osbuild-composer from sources, follow these steps: +We recommend using the latest stable Fedora for development but latest minor +release of RHEL/CentOS 8 should work just fine as well. To run osbuild-composer +from sources, follow these steps: 1. Clone the repository and create a local checkout: @@ -22,7 +26,8 @@ $ sudo dnf -y install cockpit-composer # Optional: Install cockpit i $ sudo systemctl start cockpit.socket # Optional: Start cockpit ``` -3. Finally, use the following to compile the project from the working directory, install it, and then run it: +3. Finally, use the following to compile the project from the working +directory, install it, and then run it: ``` $ make rpm @@ -30,11 +35,15 @@ $ sudo dnf -y install rpmbuild/RPMS/x86_64/* $ sudo systemctl start osbuild-composer.socket ``` -You can now open https://localhost:9090 in your browser to open cockpit console and check the "Image Builder" section. +You can now open https://localhost:9090 in your browser to open cockpit console +and check the "Image Builder" section. -Alternatively you can use `composer-cli` to interact with the Weldr API. We don't have any client for the RCM API, so the only option there is a plain `curl`. +Alternatively you can use `composer-cli` to interact with the Weldr API. We +don't have any client for the RCM API, so the only option there is a +plain `curl`. -When developing the code, use `go` executable to build, run, and test you code [1]: +When developing the code, use `go` executable to build, run, and test you +code [1]: ``` $ go test ./... @@ -48,21 +57,29 @@ See [test/README.md](test/README.md) for more information about testing. ### Planning the work -In general we encourage you to first fill in an issue and discuss the feature you would like to work on before you start. This can prevent the scenario where you work on something we don't want to include in our code. +In general we encourage you to first fill in an issue and discuss the feature +you would like to work on before you start. This can prevent the scenario where +you work on something we don't want to include in our code. -That being said, you are of course welcome to implement an example of what you would like to achieve. +That being said, you are of course welcome to implement an example of what you +would like to achieve. ### Creating a PR The commits in the PR should have these properties: * Preferably minimal and well documented - * Where minimal means: don't do unrelated changes even if the code is obviously wrong + * Where minimal means: don't do unrelated changes even if the code is + obviously wrong * Well documented: both code and commit message - * The commit message should start with the module you work on, like: `weldr: `, or `rcm:` -* The code should compile and the composer should start, so that we can run `git bisect` on it + * The commit message should start with the module you work on, + like: `weldr: `, or `rcm:` +* The code should compile and the composer should start, so that we can run + `git bisect` on it * All code should be covered by unit-tests ### Notes: -[1] If you are running macOS, you can still compile osbuild-composer. If it doesn't work out of the box, use `-tags macos` together with any `go` command, for example: `go test -tags macos ./...` +[1] If you are running macOS, you can still compile osbuild-composer. If it + doesn't work out of the box, use `-tags macos` together with any `go` + command, for example: `go test -tags macos ./...`