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

feat: pass the app_config to pre_parse_args() #541

Merged
merged 5 commits into from
Oct 22, 2024

Conversation

tigarmo
Copy link
Contributor

@tigarmo tigarmo commented Oct 21, 2024

This lets commands be able to always rely on self.config existing, even when running fill_parser() for help output.

Fixes #530

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run tox?

This lets commands be able to always rely on `self.config` existing, even
when running `fill_parser()` for help output.

Fixes #530
@tigarmo tigarmo force-pushed the work/CRAFT-3586-app-config-pre-parse branch from 517932c to 68def0c Compare October 21, 2024 20:42
pyproject.toml Outdated Show resolved Hide resolved
@tigarmo tigarmo marked this pull request as ready for review October 21, 2024 20:58
Copy link

@upils upils left a comment

Choose a reason for hiding this comment

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

LGTM, but I have a couple of questions on things I am unsure about.

docs/reference/changelog.rst Show resolved Hide resolved
craft_application/commands/base.py Show resolved Hide resolved
@tigarmo tigarmo added the squash label Oct 22, 2024
@tigarmo tigarmo merged commit 384153d into main Oct 22, 2024
9 of 10 checks passed
@tigarmo tigarmo deleted the work/CRAFT-3586-app-config-pre-parse branch October 22, 2024 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Init command cannot render a help message
4 participants