Skip to content
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.

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

chore(bolt-sidecar): fix lints #434

Merged
merged 2 commits into from
Nov 27, 2024
Merged

chore(bolt-sidecar): fix lints #434

merged 2 commits into from
Nov 27, 2024

Conversation

estensen
Copy link
Contributor

Ran cargo clippy locally and CI is running cargo clippy --all-targets --all-features -- -D warnings

@estensen estensen requested a review from merklefruit November 27, 2024 10:04
@merklefruit
Copy link
Collaborator

It would be nice to add quick recipes for linting in the justfile, something like

clippy package:
	cd {{ package }} && cargo clippy --all-targets --all-features -- -D warnings

so we can do just clippy bolt-sidecar

@estensen
Copy link
Contributor Author

It would be nice to add quick recipes for linting in the justfile, something like

clippy package:
	cd {{ package }} && cargo clippy --all-targets --all-features -- -D warnings

so we can do just clippy bolt-sidecar

Great idea!
#436

@mempirate
Copy link
Contributor

@estensen @merklefruit you could even install this in a pre-commit hook

@estensen
Copy link
Contributor Author

@estensen @merklefruit you could even install this in a pre-commit hook

yes, but let's improve it one commit at a time

@merklefruit
Copy link
Collaborator

merklefruit commented Nov 27, 2024

pre-commit hook

My personal opinion is that CI >>> pre-commit hooks.
I would greatly appreciate if we don't add any hooks to slow down our workflow

@estensen
Copy link
Contributor Author

Not sure why I'm not able to repro failures locally

rustc 1.82.0 (f6e511eec 2024-10-15)

@merklefruit
Copy link
Collaborator

Not sure why I'm not able to repro failures locally

rustc 1.82.0 (f6e511eec 2024-10-15)

same

@estensen estensen requested a review from thedevbirb November 27, 2024 11:25
@thedevbirb thedevbirb merged commit f436057 into unstable Nov 27, 2024
3 checks passed
@thedevbirb thedevbirb deleted the sidecar-lint2 branch November 27, 2024 11:28
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.

4 participants