-
-
Notifications
You must be signed in to change notification settings - Fork 388
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
Restructure cli commands #4467
Restructure cli commands #4467
Conversation
Deployment of preview was torn down |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4467 +/- ##
==========================================
- Coverage 28.22% 28.07% -0.16%
==========================================
Files 385 396 +11
Lines 27847 28001 +154
==========================================
Hits 7860 7860
- Misses 19296 19450 +154
Partials 691 691 ☔ View full report in Codecov by Sentry. |
1ba8ec4
to
6fac7ed
Compare
Could you add a migration instruction / hint to the docs? |
Sure but what do you have in mind? The new structure is covered by the docs and the change itself is listed as breaking change in the release notes already. |
I would suggest adding something like to https://woodpecker-ci.org/migrations#user-migrations
My goal before releasing 3.0 would be to have a migration list a user / admin could easily follow like a manual. |
OK. IMO that's a bit overkill and frankly very tedious, but I can look into it. |
Fixes: #3866
Personally, I would keep the
pipeline
command at the top-level but open for other suggestions. What do you think about sorting commands alphabetically? To me, the current command order looks a bit random.