-
Notifications
You must be signed in to change notification settings - Fork 12
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
Disable validation of output smithy files by default. #219
Disable validation of output smithy files by default. #219
Conversation
This makes the tool a little bit more lenient for some users. A flag is added to re-enable validation.
…of-output-smithy-files
@@ -64,7 +64,7 @@ _Note: this library is published to work on Java 8 and above. However, you will | |||
### CLI Usage | |||
|
|||
The `smithytranslate` CLI will recursively go through all child directories of the | |||
input directory provided and format any Smithy files it finds. The output | |||
input directory provided and format any Smithy files it finds. The output |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unrelated to your PR, but it seems like the output
is just a leftover incomplete sentence
@@ -123,13 +123,15 @@ Options and flags: | |||
Display this help text. | |||
--input <path>, -i <path> | |||
input source files | |||
--verboseNames |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for aligning all CLI options on kebab
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love what I see, thanks for updating the readme section of the formatter
This makes the tool a little bit more lenient for some users. A flag is added to re-enable validation.
also changes the CLI options to be kebab-cased
Todo