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

CommandLine cleanup #1568

Merged
merged 4 commits into from
Feb 9, 2022
Merged

CommandLine cleanup #1568

merged 4 commits into from
Feb 9, 2022

Commits on Feb 8, 2022

  1. Renamed CommandReflection#initFromAnnotatedFields(…) as initFromAnnot… 

    …atedMembers because it works on both fields & methods
    @rgoldberg
    rgoldberg committed Feb 8, 2022
    Configuration menu
    Copy the full SHA
    a6a22c6 View commit details
    Browse the repository at this point in the history

  2. CommandReflection#initSubcommands(…) hierarchy param is now Set<Class… 

    …<?>> instead of Stack<Class<?>> because it's only used for contains

Renamed originalHierarchy local variable as fullHierarchySet because the former sounds as if the hierarchy was (or is somehow being) changed, but it's not
    @rgoldberg
    rgoldberg committed Feb 8, 2022
    Configuration menu
    Copy the full SHA
    1f0e7ac View commit details
    Browse the repository at this point in the history

  3. Simplified CommandUserObject constructor

    @rgoldberg
    rgoldberg committed Feb 8, 2022
    Configuration menu
    Copy the full SHA
    bb822ea View commit details
    Browse the repository at this point in the history

  4. Merge branch 'master' into command-line-cleanup

    @remkop
    remkop committed Feb 8, 2022
    Configuration menu
    Copy the full SHA
    105a3df View commit details
    Browse the repository at this point in the history