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.

Sign up for GitHub

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

Jump to bottom

Support CSV format for get_info and allow including any violation types #110

Merged
merged 8 commits into from
Oct 13, 2023
Merged

Support CSV format for get_info and allow including any violation types #110

merged 8 commits into from
Oct 13, 2023

Commits on Oct 12, 2023

  1. Refactor to separate stats-collection from output (no visible change)

    @oboxodo
    oboxodo committed Oct 12, 2023
    Configuration menu
    Copy the full SHA
    33be05e View commit details
    Browse the repository at this point in the history

  2. Use data instead of methods to distinguish violation type

    @oboxodo
    oboxodo committed Oct 12, 2023
    Configuration menu
    Copy the full SHA
    7d6ca58 View commit details
    Browse the repository at this point in the history

  3. Use hashes and arrays allow easy support for new types (no visible ch… 

    …ange)
    @oboxodo
    oboxodo committed Oct 12, 2023
    Configuration menu
    Copy the full SHA
    490dd31 View commit details
    Browse the repository at this point in the history

  4. Add support for reporting packs info in CSV format

    @oboxodo
    oboxodo committed Oct 12, 2023
    Configuration menu
    Copy the full SHA
    b876ba6 View commit details
    Browse the repository at this point in the history

  5. Include architecture violations on get_info

    @oboxodo
    oboxodo committed Oct 12, 2023
    Configuration menu
    Copy the full SHA
    8c874fe View commit details
    Browse the repository at this point in the history

  6. Allow specifying what types of violations to output on get_info

    @oboxodo
    oboxodo committed Oct 12, 2023
    Configuration menu
    Copy the full SHA
    7c24364 View commit details
    Browse the repository at this point in the history

  7. Allow including a Date in the get_info report (useful for snapshots) 

    Including the date as part of the report makes it easy to store the output as snapshots over time to compare progress.
    @oboxodo
    oboxodo committed Oct 12, 2023
    Configuration menu
    Copy the full SHA
    9f419a5 View commit details
    Browse the repository at this point in the history

  8. Prefer some more code readability over DRYness 

    This is to satisfy some feedback I got on the PR.
    @oboxodo
    oboxodo committed Oct 12, 2023
    Configuration menu
    Copy the full SHA
    3431f36 View commit details
    Browse the repository at this point in the history