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

Added script to generate altsrc flag definitions #1244

Closed
wants to merge 1 commit into from
Closed

Added script to generate altsrc flag definitions #1244

wants to merge 1 commit into from

Commits on Feb 23, 2021

  1. Improved consistency in implementation and testing of file-based inpu… 

    …t sources.

- Updated unit tests for atlsrc handling to match expected usage via cli.App. Previously, the tests tried to mock cli.Context which caused differences in behavior.
- The types of altsrc flags are now limited to the subset that provide FlagInputSourceExtension implementation and have matching methods in InputSourceContext interface. Removed types where never updated from the input source.
- Added script to generate altsrc flag definitions, restricted list of types to the ones that provide implementation of FlagInputSourceExtension. The original program to generate altsrc flags is nowhere to be found, so replaced with fg.py.
    @ipostelnik
    ipostelnik committed Feb 23, 2021
    Configuration menu
    Copy the full SHA
    d3bb346 View commit details
    Browse the repository at this point in the history