Skip to content

cli exec: have uniq prefix for each workflow#6716

Merged
qwerty287 merged 2 commits into
woodpecker-ci:mainfrom
6543-forks:cli-exec_gen-uniq-prefix
Jun 14, 2026
Merged

cli exec: have uniq prefix for each workflow#6716
qwerty287 merged 2 commits into
woodpecker-ci:mainfrom
6543-forks:cli-exec_gen-uniq-prefix

Conversation

@6543

@6543 6543 commented Jun 10, 2026

Copy link
Copy Markdown
Member

preparation to have cli exec run workflows in parallel

fix taken from #6490 (6c88ea3)

@6543 6543 mentioned this pull request Jun 10, 2026
9 tasks
@woodpecker-bot

woodpecker-bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Surge PR preview deployment was removed

@6543 6543 added bug Something isn't working cli labels Jun 10, 2026
@codecov

codecov Bot commented Jun 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 44.79%. Comparing base (31dd64f) to head (ca5ea22).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6716      +/-   ##
==========================================
+ Coverage   44.75%   44.79%   +0.04%     
==========================================
  Files         436      436              
  Lines       29190    29189       -1     
==========================================
+ Hits        13063    13075      +12     
+ Misses      15023    15011      -12     
+ Partials     1104     1103       -1     

☔ View full report in Codecov by Harness.
📢 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.

@qwerty287

Copy link
Copy Markdown
Contributor

How does removing the prefix make it unique? I scanned through the compiler code but couldn't find a place where it's generated if not set

@6543

6543 commented Jun 10, 2026

Copy link
Copy Markdown
Member Author

compiler.WithPrefix(

it was just overloaded here ...

@qwerty287 qwerty287 enabled auto-merge (squash) June 14, 2026 07:17
@qwerty287 qwerty287 merged commit 54ec5b4 into woodpecker-ci:main Jun 14, 2026
6 of 7 checks passed
@woodpecker-bot woodpecker-bot mentioned this pull request Jun 14, 2026
1 task
@6543 6543 deleted the cli-exec_gen-uniq-prefix branch June 14, 2026 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working cli

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants