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

CLI config groupings #32

Merged
merged 1 commit into from
Aug 8, 2023
Merged

CLI config groupings #32

merged 1 commit into from
Aug 8, 2023

Conversation

hopeyen
Copy link
Collaborator

@hopeyen hopeyen commented Aug 3, 2023

Description

Group configurations, no changes required for the users

Issue link (if applicable)

Partial to #31

Checklist

  • Are tests up-to-date with the new changes?
  • Are docs up-to-date with the new changes? (Open PR on docs repo if necessary)

@hopeyen hopeyen added the type:refactor Changes not visible to users label Aug 3, 2023
@hopeyen hopeyen requested a review from pete-eiger August 3, 2023 00:23
@hopeyen hopeyen self-assigned this Aug 3, 2023
@hopeyen hopeyen changed the title Hope/config groupings CLI config groupings Aug 3, 2023
pete-eiger
pete-eiger previously approved these changes Aug 6, 2023
@hopeyen hopeyen linked an issue Aug 7, 2023 that may be closed by this pull request
@hopeyen hopeyen force-pushed the hope/config-groupings branch 2 times, most recently from a2f9a82 to ee04adb Compare August 7, 2023 16:26
@hopeyen hopeyen merged commit 1997e9d into dev Aug 8, 2023
6 checks passed
@hopeyen hopeyen deleted the hope/config-groupings branch August 8, 2023 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:refactor Changes not visible to users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Proper control flow
2 participants