Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Config load #13

Merged
merged 5 commits into from
Feb 1, 2022
Merged

Config load #13

merged 5 commits into from
Feb 1, 2022

Conversation

cinek810
Copy link
Owner

No description provided.

@cinek810 cinek810 force-pushed the config_load branch 6 times, most recently from d6f4831 to eb9ec9e Compare January 31, 2022 10:17
@cinek810 cinek810 requested a review from LegenJCdary January 31, 2022 10:19
@cinek810 cinek810 force-pushed the config_load branch 2 times, most recently from fb1ca3d to 8064045 Compare January 31, 2022 11:07
Comment on lines 216 to 218
subcommand = get_sub_command(sys.argv[1])
options = parse_options(sys.argv[2:])
validate_options(options, subcommand)
config = load_configuration()
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ja bym te rzeczy do maina wyciągnął.

@cinek810 cinek810 force-pushed the config_load branch 3 times, most recently from ee8a48d to eb9ec9e Compare January 31, 2022 12:20
@cinek810 cinek810 force-pushed the config_load branch 2 times, most recently from 28e12ac to 145466e Compare January 31, 2022 12:27
@cinek810 cinek810 force-pushed the config_load branch 3 times, most recently from 756ac1e to 0461e54 Compare January 31, 2022 19:03
TODO: enable this check when we have basic functionality
@cinek810 cinek810 mentioned this pull request Jan 31, 2022
@LegenJCdary LegenJCdary merged commit e8cf8a3 into main Feb 1, 2022
@LegenJCdary LegenJCdary deleted the config_load branch February 1, 2022 08:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants