Skip to content

Commit

Permalink
feat(build): fluidattacks#1108 document contributing
Browse files Browse the repository at this point in the history
Add new sections to the contributing.md file to help newcomers.

- Running your local changes
- Adding yourself to the mailmap
- Validating commit message
- DCO
- Adding and rendering documentation

Update lintMarkdown config.

Signed-off-by: Robin Quintero <rohaquinlop301@gmail.com>
  • Loading branch information
rohaquinlop committed Jun 17, 2023
1 parent 97ccd84 commit 68194ce
Show file tree
Hide file tree
Showing 2 changed files with 48 additions and 1 deletion.
47 changes: 47 additions & 0 deletions docs/src/contributing.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,53 @@ Examples:
- [feat(build): #252 aws secrets from env](https://github.com/fluidattacks/makes/commit/1c9f06a809bd92d56939d5809ce46058856fdf0a)
- [feat(build): #232 make parallel utils](https://github.com/fluidattacks/makes/commit/99e9f77482a6cbc9858a7a928a91a8a8aa9ff353)

### Testing your local changes

You can run local changes by simply running `m . <job>`.
If you're adding new files, make sure to `git add` them so Makes can recognize
them.

### Adding yourself to the mailmap

When contributing is important to add yourself to the
[.mailmap](https://github.com/fluidattacks/makes/blob/main/.mailmap) file.
Make sure to test it with `m . /lintGitMailMap`.

### Validating commit message

- Your commit message must follow this [syntax](https://github.com/fluidattacks/makes/tree/main/test/lint-commit-msg).
- You can take a look at other commits [here](https://github.com/fluidattacks/makes/commits/main).
- Make sure to test it with `m . /lintGitCommitMsg`.

### DCO

You must sign your commits by adding a `Signed-off-by` line at the end of your
commit messages.

Example:

```
feat(build): #1099 licensing builtin with reuse
- Create `testLicense` builtin with reuse.
- Add job to dev pipeline
Signed-off-by: Robin Quintero <rohaquinlop301@gmail.com>
```

### Adding documentation

All added features must be documented in the same PR.
You can run `m . /docs/dev` to serve the [doc site](https://makes.fluidattacks.com)
on localhost.

### Other PR rules

A PR must:

- Only be one commit ahead of main.
- Have a title and body equal to its commmit message.

### The legal side of contributions

All of the code
Expand Down
2 changes: 1 addition & 1 deletion test/lint-markdown/config.rb
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
rule 'MD010'
rule 'MD011'
rule 'MD012'
rule 'MD013', :tables => false, :code_blocks => false
rule 'MD013', :tables => false, :ignore_code_blocks => true
rule 'MD018'
rule 'MD019'
rule 'MD020'
Expand Down

0 comments on commit 68194ce

Please sign in to comment.