Ensure cli exec has by default not the same prefix

[6543]

this prohibits collisions

this make it similar to the server:

woodpecker/server/pipeline/stepbuilder/stepBuilder.go

Lines 289 to 295 in eebaa10

	compiler.WithPrefix(
	fmt.Sprintf(
	"wp_%s_%d",
	strings.ToLower(ulid.Make().String()),
	workflowID,
	),
	),

[woodpecker-bot]

Tearing down https://woodpecker-ci-woodpecker-pr-4132.surge.sh

[anbraten]

I would suggest to simply hard-code it in the cli to the same value we use in the agent. wp_%s_%d.

[6543]

I would suggest to simply hard-code it in the cli to the same value we use in the agent. wp_%s_%d.

no hardcode will make colisins as we have right now ... but I could change the format to have a wp_ prefix. the problem is that we dont have a number that is even near the truth for each workflow

[anbraten]

no hardcode will make colisins as we have right now ... but I could change the format to have a wp_ prefix. the problem is that we dont have a number that is even near the truth for each workflow

ulid.Make().String() is fine. I meant we don't need a cli option for it at all as users can't change it for the agent as well.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 26.18%. Comparing base (b59533a) to head (c2be418).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4132      +/-   ##
==========================================
- Coverage   26.20%   26.18%   -0.02%     
==========================================
  Files         373      373              
  Lines       27034    27034              
==========================================
- Hits         7084     7079       -5     
- Misses      19294    19299       +5     
  Partials      656      656              

Flag	Coverage Δ
	26.18% <ø> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[6543]

well you normally dont need it but why remove it ... if we should not present this to the user we can just hide that flag

[6543]

*it's alrady hidden so why remove it etc ?!?

[anbraten]

*it's alrady hidden so why remove it etc ?!?

Why do we need to support changing it? It's just extra code ...

[6543]

Well for those who wana change it at some point could also just compile the cli anyway ...

[6543]

@anbraten done