Skip to content

Commit

Permalink
ci: unify fmt check (#3159)
Browse files Browse the repository at this point in the history
## What ❔

prover ci should check formatting with zkstack_cli 

## Why ❔

<!-- Why are these changes done? What goal do they contribute to? What
are the principles behind them? -->
<!-- Example: PR templates ensure PR reviewers, observers, and future
iterators are in context about the evolution of repos. -->

## Checklist

<!-- Check your PR fulfills the following items. -->
<!-- For draft PRs check the boxes as you complete them. -->

- [ ] PR title corresponds to the body of PR (we generate changelog
entries from PRs).
- [ ] Tests for the changes have been added / updated.
- [ ] Documentation comments have been added / updated.
- [ ] Code has been formatted via `zkstack dev fmt` and `zkstack dev
lint`.
  • Loading branch information
perekopskiy authored Oct 23, 2024
1 parent d0f61b0 commit 4629450
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion .github/workflows/ci-prover-reusable.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,9 @@ jobs:
ci_run zkstack dev db setup --prover-url=${{ env.prover_url }} --core-url=${{ env.core_url }}
- name: Formatting
run: ci_run bash -c "cd prover && cargo fmt --check"
run: |
ci_run git config --global --add safe.directory /usr/src/zksync
ci_run zkstack dev fmt --check rustfmt
unit-tests:
runs-on: [ matterlabs-ci-runner-highmem-long ]
Expand Down

0 comments on commit 4629450

Please sign in to comment.