Skip to content

Commit

Permalink
Undo aarch64 rustfmt skips.
Browse files Browse the repository at this point in the history
These skips were added in a-scie#173 to work around this issue:
  rust-lang/rustfmt#5964

It has been resolved; so formatting should now work on all
architectures.

Fixes a-scie#174
  • Loading branch information
jsirois committed Dec 9, 2023
1 parent ab1e968 commit 5cd7986
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 9 deletions.
10 changes: 4 additions & 6 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,12 +26,10 @@ jobs:
resource_class: arm.medium
steps:
- checkout
# TODO(John Sirois): https://github.com/a-scie/jump/issues/174
# Undo once https://github.com/rust-lang/rustfmt/issues/5964 is fixed.
#- rust/install:
# version: nightly
#- rust/format:
# nightly-toolchain: true
- rust/install:
version: nightly
- rust/format:
nightly-toolchain: true
- rust/install
- rust/clippy
- rust/test:
Expand Down
3 changes: 0 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,6 @@ jobs:
steps:
- uses: actions/checkout@v3
- name: Check Formatting
# TODO(John Sirois): https://github.com/a-scie/jump/issues/174
# Undo once https://github.com/rust-lang/rustfmt/issues/5964 is fixed.
if: matrix.os != 'macos-13-aarch64'
run: |
rustup toolchain add nightly -c rustfmt
cargo +nightly fmt --check --all
Expand Down

0 comments on commit 5cd7986

Please sign in to comment.