Skip to content

Document the manual error check for repo version#199

Merged
moroten merged 2 commits into
mainfrom
feature/document-the-manual-error-check-for-repo-version
Jun 18, 2026
Merged

Document the manual error check for repo version#199
moroten merged 2 commits into
mainfrom
feature/document-the-manual-error-check-for-repo-version

Conversation

@stagnation

@stagnation stagnation commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

This mainly improves the code around the manual error check for curl. Where curl's own message is verbose progress information combined with a suboptimal message on errors: "curl: (22) The requested URL returned error: 422".
During development we had a "huh, that is weird" and realized that subshells do not inherit the error exit settings. We do use subshells rather than ouptut variables for function calls.

@stagnation stagnation requested a review from oscar-meroton June 18, 2026 08:58
@stagnation stagnation force-pushed the feature/document-the-manual-error-check-for-repo-version branch from b0ae4cd to e3fa11c Compare June 18, 2026 08:59
@oscar-meroton

Copy link
Copy Markdown
Collaborator

lgtm, thanks for looking into this!

@moroten moroten merged commit 320ff9c into main Jun 18, 2026
3 checks passed
@moroten moroten deleted the feature/document-the-manual-error-check-for-repo-version branch June 18, 2026 22:12
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.

3 participants