Skip to content

Fix clippy workflow #40

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 13 commits into from
Jun 17, 2025
Merged

Fix clippy workflow #40

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
aa20da4
try fix clippy workflow
DamienVoreiter Jun 17, 2025
3399fa2
weird
DamienVoreiter Jun 17, 2025
a1a0a41
better
DamienVoreiter Jun 17, 2025
e138117
deps
DamienVoreiter Jun 17, 2025
9ff6b88
add deps
DamienVoreiter Jun 17, 2025
2e171b5
try without env
DamienVoreiter Jun 17, 2025
a7a8862
fix clippy
DamienVoreiter Jun 17, 2025
06ddfe2
try remove deps
DamienVoreiter Jun 17, 2025
66d3a93
try remove deps
DamienVoreiter Jun 17, 2025
cc4310e
try remove deps
DamienVoreiter Jun 17, 2025
31967b1
try remove deps
DamienVoreiter Jun 17, 2025
8fc4763
try remove deps
DamienVoreiter Jun 17, 2025
e531169
ok
DamienVoreiter Jun 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 17 additions & 6 deletions .github/workflows/clippy.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ on: push

env:
CARGO_TERM_COLOR: always
LIBCLANG_PATH: /usr/lib/llvm-14/lib/

jobs:
clippy_check:
Expand All @@ -14,9 +13,21 @@ jobs:
version: [community, enterprise]
steps:
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
- run: rustup component add clippy
- uses: actions-rs/clippy-check@v1

- name: Install deps
run: |
sudo apt-get update -y
sudo apt-get install -y --no-install-recommends libpcsclite-dev libev-dev libevent-dev libpcsclite-dev pkg-config clang llvm llvm-dev libclang-dev

- name: Install rust
uses: dtolnay/rust-toolchain@888c2e1ea69ab0d4330cbf0af1ecc7b68f368cc1 # v1
with:
token: ${{ secrets.GITHUB_TOKEN }}
name: Clippy
args: --features=${{ matrix.version }}
toolchain: stable
components: clippy

- name: Clippy check
shell: bash
run: |
set +e
# shellcheck disable=SC2046
cargo clippy --color always --features=${{ matrix.version }}
6 changes: 3 additions & 3 deletions src/encryptable.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,9 @@ use crate::{
/// * The original key will be prefixed with 'encrypted$'.
///
/// * The transformed Encryptable dictionary will contain `alg` property indicating
/// the encryption algorithm, `ciphertext` property whose value is a base-64 string of the
/// encrypted value, and optionally `kid` property indicating the encryption key identifier
/// if specified when returning the result of PropertyEncryptor callback call.
/// the encryption algorithm, `ciphertext` property whose value is a base-64 string of the
/// encrypted value, and optionally `kid` property indicating the encryption key identifier
/// if specified when returning the result of PropertyEncryptor callback call.
///
/// For security reason, a document that contains Encryptable dictionaries will fail
/// to push if their value cannot be encrypted including
Expand Down