Skip to content

cli: show description of default value for --backend-local-temp-dir instead of value#5656

Merged
6543 merged 4 commits intowoodpecker-ci:mainfrom
6543-forks:cli-docs-show-no-random-tmp-dir-as-description
Oct 20, 2025
Merged

cli: show description of default value for --backend-local-temp-dir instead of value#5656
6543 merged 4 commits intowoodpecker-ci:mainfrom
6543-forks:cli-docs-show-no-random-tmp-dir-as-description

Conversation

@6543
Copy link
Copy Markdown
Member

@6543 6543 commented Oct 19, 2025

blocked by urfave/cli#2210 upstream fixed

before

at https://woodpecker-ci.org/docs/cli#exec :
image

now

image

@6543 6543 added bug Something isn't working documentation docu & docs blocked something external is blocking it labels Oct 19, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 21.29%. Comparing base (deadc0e) to head (230e092).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5656      +/-   ##
==========================================
- Coverage   21.31%   21.29%   -0.02%     
==========================================
  Files         420      420              
  Lines       38080    38080              
==========================================
- Hits         8116     8111       -5     
- Misses      29211    29216       +5     
  Partials      753      753              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@6543 6543 removed the blocked something external is blocking it label Oct 19, 2025
@6543 6543 marked this pull request as ready for review October 19, 2025 11:24
@woodpecker-bot
Copy link
Copy Markdown
Contributor

woodpecker-bot commented Oct 19, 2025

Surge PR preview deployment was removed

@6543
Copy link
Copy Markdown
Member Author

6543 commented Oct 20, 2025

fixed bug upstream: urfave/cli-docs#17 ... let's see how long it takes to get merged

@6543 6543 requested a review from a team October 20, 2025 17:14
@6543 6543 merged commit 75f7730 into woodpecker-ci:main Oct 20, 2025
8 of 9 checks passed
@6543 6543 deleted the cli-docs-show-no-random-tmp-dir-as-description branch October 20, 2025 21:30
@woodpecker-bot woodpecker-bot mentioned this pull request Oct 20, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation docu & docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants