Skip to content

[Enhancement]: cfn-guard as a library - #458

Merged
joshfried-aws merged 13 commits into
aws-cloudformation:mainfrom
joshfried-aws:main
Feb 15, 2024
Merged

joshfried-aws merged 13 commits into
aws-cloudformation:mainfrom
joshfried-aws:main

Conversation

@joshfried-aws

@joshfried-aws joshfried-aws commented Feb 14, 2024 •

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:

  • Migrate from clap builder to clap derive
  • Expose all 4 cfn-guard commands as a library, allowing users to build them via a builder
  • Refactor main method
  • Bump up from clap 4.1.4 -> 4.5

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@dannyvassallo dannyvassallo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is so much more terse. I love it.

Comment thread guard/src/commands/files.rs Resolved
Comment thread guard/src/commands/completions.rs Resolved
Comment thread guard/src/commands/mod.rs Resolved
Comment thread guard/src/commands/parse_tree.rs Resolved
Comment thread guard/src/commands/parse_tree.rs Resolved
Comment thread guard/src/commands/rulegen.rs Resolved
Comment thread guard/src/main.rs Resolved
@joshfried-aws
joshfried-aws merged commit cf160dc into aws-cloudformation:main Feb 15, 2024
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.

3 participants