Skip to content

fix(ci): bump Docker Rust image to 1.88 and fix Clippy lints for Solana macros#116

Open
joaquinbejar wants to merge 2 commits into
mainfrom
chore/update-all-dependencies
Open

fix(ci): bump Docker Rust image to 1.88 and fix Clippy lints for Solana macros#116
joaquinbejar wants to merge 2 commits into
mainfrom
chore/update-all-dependencies

Conversation

@joaquinbejar
Copy link
Copy Markdown
Owner

Fixes CI failures on main after dependency update merge.

Changes

  1. Docker Rust image 1.85 → 1.88cargo-chef's dependency cargo-platform@0.3.2 requires rustc ≥ 1.88. All 3 Dockerfiles updated.

  2. Clippy CI split — The #[program] macro from Anchor/Solana generates code with unexpected_cfgs and diverging_sub_expression warnings that fail under -D warnings. Now runs in two steps:

    • Workspace (excluding program): strict -D warnings
    • Program crate: -D warnings with upstream Solana macro lints allowed

CI Actions:
- docker/setup-buildx-action v3 → v4
- docker/build-push-action v6 → v7
- docker/setup-qemu-action v3 → v4
- docker/login-action v3 → v4
- docker/metadata-action v5 → v6

Rust dependencies:
- anchor-lang 0.32 → 1.0 (breaking: remove explicit enum discriminants, CpiContext takes Pubkey, Context single lifetime)
- anchor-spl 0.32 → 1.0
- solana-program 3.0 → 4.0
- solana-sdk 3.0 → 4.0
- tokio-tungstenite 0.28 → 0.29
- axum-test 18.7 → 20.0
- deadpool-redis 0.22 → 0.23
- anchor-cli 0.32.1 → 1.0.0 (CI)

TypeScript SDK:
- @eslint/js 9.39.2 → 10.0.1
- eslint 9.39.2 → 10.0.0
- @types/node 25.2.0 → 25.5.0
- typescript-eslint 8.54.0 → 8.57.0
- vitest 4.0.18 → 4.1.0
- Remove redundant @typescript-eslint/eslint-plugin and @typescript-eslint/parser

Closes #98 #100 #101 #102 #104 #105 #106 #107 #108 #110 #111 #112 #113 #114
…na macros

- Bump rust:1.85-bookworm → rust:1.88-bookworm in all 3 Dockerfiles
  (cargo-chef's cargo-platform@0.3.2 requires rustc ≥ 1.88)
- Split Clippy CI step: workspace with -D warnings, program with
  allowed upstream Solana/Anchor macro lints (unexpected_cfgs,
  diverging_sub_expression)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant