-
-
Notifications
You must be signed in to change notification settings - Fork 384
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make local backend work with cli exec
#4102
base: main
Are you sure you want to change the base?
Make local backend work with cli exec
#4102
Conversation
Deployment of preview was successful: https://woodpecker-ci-woodpecker-pr-4102.surge.sh |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4102 +/- ##
==========================================
- Coverage 28.29% 28.27% -0.02%
==========================================
Files 399 399
Lines 28156 28175 +19
==========================================
Hits 7966 7966
- Misses 19484 19503 +19
Partials 706 706 ☔ View full report in Codecov by Sentry. |
@anbraten if you know an better way to do:
to test local backend via cli in windows / linux please tell :) |
Will do some testing. Fyi #3967 might be quite interesting for you. |
Yes thus might allow more :) Should i hide the flag and undocument it so we have it working now and can improve it later? |
The usage of the
https://github.com/woodpecker-ci/woodpecker/compare/main...anbraten:local-path?expand=1 |
I looked at your solution, this has 2 problems:
the only thing i could addopt in my patch from your suggestion would be 1. witch breaks the convention ... |
what i could do and did, is to hide that option from the user: 7101cdb |
@anbraten are you still blocking this? |
Currently if you try to exec a pipeline via the cli tool and use the local backend, all files are missing.
This will detect this specific case and use the new added option in the local backend to specify a execution directory upfront.
Sponsored by Kithara Software GmbH