Skip to content

Commit

Permalink
ci: add comment when autofix is invoked (cataclysmbnteam#3395)
Browse files Browse the repository at this point in the history
* ci: add comment when autofix is invoked

see: autofix-ci/action#7

* Apply suggestions from code review

Co-authored-by: Olanti <olanti-p@yandex.ru>

---------

Co-authored-by: Olanti <olanti-p@yandex.ru>
  • Loading branch information
scarf005 and olanti-p authored Oct 10, 2023
1 parent 2a46a92 commit 5de9076
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 8 deletions.
13 changes: 12 additions & 1 deletion .github/workflows/autofix.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,18 @@ jobs:
- name: json formatting
run: make style-all-json-parallel RELEASE=1

- uses: autofix-ci/action@8caa572fd27b0019a65e4c695447089c8d3138b9
- uses: autofix-ci/action@d3e591514b99d0fca6779455ff8338516663f7cc
if: ${{ always() }}
with:
commit-message: "style(autofix.ci): automated formatting"
commment: |
The Autofix app has automatically formatted this Pull Request.
If you edit your PR on web UI, you can ignore this message.
If you edit your PR locally, YOU MUST DO EITHER OF THE FOLLOWING:
- Run `git pull` to merge the automated commit into your local copy of the PR branch.
- [Format your code locally](https://docs.cataclysmbn.org/en/contribute/contributing/#code-style), and force push to your PR branch.
If you don't do this, your following work will be based on the old commit, and cause MERGE CONFLICT.
7 changes: 0 additions & 7 deletions doc/src/content/docs/en/mod/json/explanation/json_style.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,6 @@ sidebar:
variant: caution
---

:::caution

[A PR may change current JSON style](https://github.com/cataclysmbnteam/Cataclysm-BN/pull/3118) in
favor of better tooling.

:::

Like in [C++ Code Style](../../../dev/explanation/code_style.md), the JSON styling policy is to
update JSON as it is added or edited, and in relatively small chunks otherwise in order to prevent
undue disruption to development.
Expand Down

0 comments on commit 5de9076

Please sign in to comment.