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

Validate only changed #25

Merged
merged 12 commits into from
Mar 15, 2024
Merged

Validate only changed #25

merged 12 commits into from
Mar 15, 2024

Conversation

wwoytenko
Copy link
Contributor

The changes are as follows:

  • The validation command was rewritten almost from scratch. Now the code is much cleaner than before
  • New option transformed-only - displays only bells that are transformed and PK
  • JSON output format is different for --diff and without it
  • The format parameter now controls the output type JSON or text
  • Added the table-format parameter which is responsible for the vertical and horizontal table orientation
  • Added the --warnings parameter, if it is specified then not only fatal-warnings will be displayed, but also those with a lower severity
  1. Diff + json + transformed-only + warnings
greenmask --config config.yml validate --warnings --data --diff --format json --transformed-only
  1. Similar to the previous one, but tabular output
greenmask --config config.yml validate --warnings --data --diff --format text --table-format vertical --transformed-only
  1. Just validation that there are no errors in the config
greenmask --config config.yml validate
  1. Printed only validation warnings
greenmask --config config.yml validate --warnings

@wwoytenko wwoytenko requested review from tony2001 and gracingpro March 6, 2024 17:24
@wwoytenko wwoytenko self-assigned this Mar 6, 2024
@wwoytenko wwoytenko requested a review from tony2001 March 7, 2024 17:10
@wwoytenko wwoytenko merged commit e549611 into main Mar 15, 2024
15 checks passed
@wwoytenko wwoytenko deleted the validate_only_changed branch March 15, 2024 21:56
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