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

Move lockfile to its own module and general refactor #29

Merged
merged 3 commits into from
May 28, 2024

Conversation

mark-thm
Copy link
Contributor

Issue

Closes #27.

rules_multitool suggests using a $schema field but this CLI doesn't support that field.

Summary

Refactor lockfile handling to its own module, update struct names for legibility, and add support for specifying $schema. Additionally, validate $schema matches expectation, always serialize a value for $schema, and add some debug prints for tool updates. Last, move regex initialization into a static block.

@mark-thm mark-thm requested review from reviewbot-theorem and a team as code owners May 28, 2024 00:28
@thm-automation thm-automation bot enabled auto-merge (squash) May 28, 2024 00:37
@mark-thm mark-thm changed the title Move lockfile to its own module Move lockfile to its own module and general refactor May 28, 2024
@thm-automation thm-automation bot merged commit cde0477 into main May 28, 2024
8 checks passed
@thm-automation thm-automation bot deleted the me/refactor branch May 28, 2024 01:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to deserialize lockfile
2 participants