Skip to content

🛡️introduce extremely strict checks and rules#126

Merged
renardeinside merged 14 commits intomainfrom
chore/strict-checks
Feb 25, 2026
Merged

🛡️introduce extremely strict checks and rules#126
renardeinside merged 14 commits intomainfrom
chore/strict-checks

Conversation

@renardeinside
Copy link
Collaborator

Summary

  • Move repeated #![forbid(...)] / #![deny(...)] lint attributes from 8 source files into a single [workspace.lints] section in root Cargo.toml, with each crate inheriting via [lints] workspace = true
  • Split pre-commit clippy into two passes: strict for lib+bins, relaxed (-A unwrap-used/expect-used/panic) for test targets — so test code doesn't need scattered #[allow] overrides
  • Refactor build scripts (core/build.rs, apx/build.rs) and studio/src/main.rs to use Result + ? instead of unwrap/expect/panic
  • Add #[derive(Debug)] to 3 databricks_sdk structs now covered by missing_debug_implementations

Test plan

  • cargo check passes
  • cargo clippy passes (both lib+bins and test targets)
  • cargo test — all 388 tests pass
  • Pre-commit hook passes end-to-end

🤖 Generated with Claude Code

@renardeinside renardeinside changed the title Centralize lint config into workspace-level Cargo.toml 🛡️introduce extremely strict checks and rules Feb 25, 2026
@renardeinside renardeinside merged commit 5ebf720 into main Feb 25, 2026
3 checks passed
@renardeinside renardeinside deleted the chore/strict-checks branch February 25, 2026 18:07
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