Skip to content

Commit

Permalink
docs(CONTRIBUTING.md): update best practices (openwall#640)
Browse files Browse the repository at this point in the history
Change notes on the intended formatting.

Signed-off-by: Claudio André <[email protected]>
  • Loading branch information
claudioandre-br committed Nov 21, 2024
1 parent 7cb1acf commit fbe5b31
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .github/actions/spelling/expect.txt
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ Acked
adxcrypt
AESNI
afl
AKIAIOSFODNN
Altivec
amazonaws
amd
Expand Down Expand Up @@ -122,6 +121,7 @@ img
IMO
intrinsics
ipc
JAVASCRIPT
jscpd
JTR
jtrcrackers
Expand Down
15 changes: 10 additions & 5 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -65,18 +65,23 @@ However, there are some cases where the rule must be overcome by common sense.

## Create a suitable PR

- Run the `pre-commit run --all-files` command.
- Format all edited or new shell scripts using shfmt.
- Fix all issues detected by markdownlint Visual Studio Code extension (davidanson.vscode-markdownlint).
- Format all edited YAML or Markdown files using "Prettier (esbenp.prettier-vscode).
- Verify to see if PR checks have passed, or if there are any new or attention-worthy warning;
- Use the linter to check for issues and correct them:

```bash
docker run --rm -e DEFAULT_BRANCH=main -e RUN_LOCAL=true -e USE_FIND_ALGORITHM=true \
-e FIX_JSON_PRETTIER=true -e FIX_MARKDOWN_PRETTIER=true -e FIX_JAVASCRIPT_PRETTIER=true \
-v $(pwd):/tmp/lint ghcr.io/super-linter/super-linter:v7.2.0 # Update the version
```

- Update checksums using the command:

```bash
cd scripts && sha256sum ./*.sh > ../requirements.hash && cd - && \
cd patches && sha256sum ./* >> ../requirements.hash && cd -
```

- Submit your change and then check if the PR checks have passed, or if there are any new or attention-worthy warnings.

## The PR review process

After you open a pull request, you as the author are responsible for resolving:
Expand Down

0 comments on commit fbe5b31

Please sign in to comment.