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

Moar (ok some) tests #23

Merged
merged 2 commits into from
Mar 19, 2024
Merged

Moar (ok some) tests #23

merged 2 commits into from
Mar 19, 2024

Conversation

yaleman
Copy link
Member

@yaleman yaleman commented Mar 18, 2024

Mainly so dependabot has a chance of flagging things when an import's wrong. I broke out some internals from main.rs into a lib so I can import them, and added runs in CI to check that clap is happy (because it panics when things are wrong at runtime, but not at compile time)

Checklist

  • This pr contains no AI generated code
  • cargo fmt has been run
  • cargo clippy has been run
  • cargo test has been run and passes
  • book chapter included (if relevant)
  • design document included (if relevant)

@yaleman yaleman requested a review from Firstyear March 18, 2024 10:12
@yaleman yaleman enabled auto-merge (squash) March 18, 2024 10:12
@yaleman yaleman merged commit 35d0e35 into kanidm:main Mar 19, 2024
4 checks passed
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