Skip to content

Commit

Permalink
Small cleanups and fixups
Browse files Browse the repository at this point in the history
- Moves bins into standard `src/bin` folder
- Adds a basic test (TODO: doesn't work on gh-actions CI)
- Reformat with new `rustfmt.toml`, rearrange some code, add some comments
- Fix wrong error message for `GetVersionExA`
- `cargo update`
  • Loading branch information
MingweiSamuel committed Aug 20, 2024
1 parent cc29e9e commit e1baeb4
Show file tree
Hide file tree
Showing 9 changed files with 716 additions and 520 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/blondie.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,9 @@ jobs:
- name: Build
run: cargo build --release --all-features --verbose

- name: Run tests
run: cargo test --release --all-features --verbose -- --nocapture
# TODO: Doesn't record any tracing events on github-actions
# - name: Run tests
# run: cargo test --release --all-features --verbose -- --nocapture

docs:
runs-on: windows-latest
Expand Down
19 changes: 1 addition & 18 deletions .github/workflows/clippy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,23 +13,6 @@ jobs:
- uses: actions/checkout@v2

- run: rustup component add clippy

# Workaround from https://github.com/actions-rs/clippy-check/issues/2#issuecomment-807878478
# - name: Check workflow permissions
# id: check_permissions
# uses: scherermichael-oss/[email protected]
# with:
# required-permission: write
# env:
# GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
# - name: Run clippy action to produce annotations
# uses: actions-rs/clippy-check@v1
# if: steps.check_permissions.outputs.has-permission
# with:
# args: --all-targets -- -D warnings
# token: ${{ secrets.GITHUB_TOKEN }}
# - name: Run clippy manually without annotations
# if: ${{ !steps.check_permissions.outputs.has-permission }}
# run: cargo clippy --all-targets -- -D warnings

- name: Run clippy manually without annotations
run: cargo clippy --all-targets -- -D warnings
Loading

0 comments on commit e1baeb4

Please sign in to comment.