Skip to content

handle parsing errors and describe document changes - #5

Merged
wagner-intevation merged 5 commits into
mainfrom
fix/1-argument-traceback
Oct 1, 2026
Merged

wagner-intevation merged 5 commits into
mainfrom
fix/1-argument-traceback

Conversation

@wagner-intevation

@wagner-intevation wagner-intevation commented Oct 1, 2026 •

Copy link
Copy Markdown
Member

and show instead the usage and error message
fixes #1

Refactor the publisher argument parsing

feat: describe changes in summary
fixes #2

For the latter, the other two changes are required. That's why a single PR contains all of them.

and show instead the usage and error message

fixes #1
Same pattern as for the other parameters
one function for parsing the parameters
one function for actually applying the changes
@wagner-intevation wagner-intevation changed the title fix: for argument errors, hide the traceback handle parsing errors and describe documetn changes Oct 1, 2026
@wagner-intevation wagner-intevation changed the title handle parsing errors and describe documetn changes handle parsing errors and describe document changes Oct 1, 2026

@JanHoefelmeyer JanHoefelmeyer left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM & works

@wagner-intevation
wagner-intevation force-pushed the fix/1-argument-traceback branch from 6f5a820 to 8699b73 Compare October 1, 2026 17:15
@wagner-intevation
wagner-intevation merged commit 2edc738 into main Oct 1, 2026
4 checks passed
@wagner-intevation
wagner-intevation deleted the fix/1-argument-traceback branch October 1, 2026 17:17
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.

Describe the made changes in the revision history Argument errors: Hide traceback

2 participants