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

Pl 132755 refactor module and subcommand structure #70

Merged
merged 25 commits into from
Aug 12, 2024

Conversation

ctheune
Copy link
Member

@ctheune ctheune commented Aug 12, 2024

Related issue(s): PL-132755, ...

  • Change is documented in changelog

The overall test coverage is a bit worse at the moment and we're likely broken on master. However, we need this as the foundational work for S3 and I'd rather not spend more time gold-plating something we don't yet know 100% we're going to use it in this shape or need to improve the shape further anyways.

Security implications

We are calling more sub-commands now, so that needs to be managed, but that's part of deploying it and normally handled with NixOS tools.

  • Security requirements tested? (EVIDENCE)

Nothing specific to test.

@ctheune
Copy link
Member Author

ctheune commented Aug 12, 2024

Overriding review as this code was originally written by @dhnasa and I'm doing the review. GitHubs little brain can't cope with that, though.

@ctheune ctheune merged commit e558a31 into main Aug 12, 2024
3 checks passed
@ctheune ctheune deleted the PL-132755-refactor-module-and-subcommand-structure branch August 13, 2024 06:50
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