Skip to content

Commit

Permalink
Add note on keeping lint settings in sync
Browse files Browse the repository at this point in the history
  • Loading branch information
fwcd committed Sep 7, 2024
1 parent 816697d commit f355c62
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
4 changes: 4 additions & 0 deletions .vscode/settings.json
Original file line number Diff line number Diff line change
@@ -1,4 +1,8 @@
{
// NOTE: This should be kept in sync with scripts/lint. Unfortunately it looks
// like we cannot just let rust-analyzer invoke our script, so we'll have to
// sync this manually until we figure out how to configure clippy directly
// (e.g. via the Cargo.toml).
"rust-analyzer.check.command": "clippy",
"rust-analyzer.check.extraArgs": [
"--",
Expand Down
5 changes: 5 additions & 0 deletions scripts/lint
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,11 @@
set -e
cd "$(dirname $0)/.."

# NOTE: This should be kept in sync with .vscode/settings.json. Unfortunately it
# looks like we cannot just let rust-analyzer invoke our script, so we'll have
# to sync this manually until we figure out how to configure clippy directly
# (e.g. via the Cargo.toml).

# TODO: Figure out if we can configure this properly from Cargo.toml
# [lints.clippy] seems to be unstable currently.

Expand Down

0 comments on commit f355c62

Please sign in to comment.