Skip to content

Conversation

@nyurik
Copy link
Member

@nyurik nyurik commented Nov 25, 2025

Most of these changes are to make this project share the same settings as all other OxiBUS crates.

  • configure precommit.ci
  • some deny, codecov, and clippy configurations - some unused at this point, but will be enabled in CI shortly
  • modify .editorconfig to insert a final newline -- this is a much more common setting, and corresponds to the default precommit settings everywhere
  • ignore fmt for the autogenerated messages file in two crates
  • some clippy fixes

@nyurik nyurik force-pushed the linting branch 2 times, most recently from 96d22bf to 07563a6 Compare November 26, 2025 18:58
Most of these changes are to make this project share the same settings as all other OxiBUS crates.

* configure [precommit.ci](https://pre-commit.com/)
* add [justfile](https://github.com/casey/just?tab=readme-ov-file) - common recipes same as for all other projects
* some deny, codecov, and clippy configurations - some unused at this point, but will be enabled in CI shortly
* modify .editorconfig to insert a final newline -- this is a much more common setting, and corresponds to the default precommit settings everywhere
* run `cargo fmt` with some extra settings to clean up the `use` statements
* ignore fmt for the autogenerated messages file in two crates
Copy link
Member

@tegimeki tegimeki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@nyurik nyurik force-pushed the linting branch 2 times, most recently from 64875b1 to 9c1a302 Compare November 28, 2025 07:01
@nyurik nyurik merged commit 0a2b1f4 into oxibus:main Nov 28, 2025
4 checks passed
@nyurik nyurik deleted the linting branch November 28, 2025 07:03
@nyurik nyurik mentioned this pull request Jan 20, 2026
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.

2 participants