Skip to content

Conversation

@fernandrone
Copy link
Contributor

Reverts #196

Quoting from @silverwind on #196

This breaks existing use with relative paths like foo/ which fail with strip_prefix must start with '/'.

@fernandrone
Copy link
Contributor Author

fernandrone commented Sep 2, 2025

@Ompragash this is breaking several of our pipelines which are pulling from :latest (and likely breaking other users too).

Given the issue I'd suggest reverting this.

cc @DevanshMathur19

@Ompragash Ompragash merged commit d50ece1 into drone-plugins:master Sep 2, 2025
6 checks passed
@Ompragash
Copy link
Member

Thanks, @fernandrone, for quickly creating a revert PR to fix the problem caused by the earlier merge. Your prompt assistance is greatly appreciated!

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.

2 participants