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

Fix command line argument names #70

Merged
merged 4 commits into from
Jan 24, 2022
Merged

Fix command line argument names #70

merged 4 commits into from
Jan 24, 2022

Conversation

gicmo
Copy link
Contributor

@gicmo gicmo commented Jan 24, 2022

Also include openssl in the dev container and move the pylint configuration to setup.cfg.

This is what we do in osbuild, lets do it here too, so we are in
sync about where the configuration is.
Fix the names for various command line arguments for when the
builder plugin is invoked as a stand-alone executable.
Needed for certificate generation.
Two lines before the first class. Will make auto-format happy;
and David.
Copy link
Member

@teg teg left a comment

Choose a reason for hiding this comment

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

This seems trivial enough that even I can approve ;)

@gicmo gicmo merged commit 9c5f38f into osbuild:main Jan 24, 2022
@gicmo gicmo deleted the cmd_args_names branch January 24, 2022 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants