woodpecker
woodpecker copied to clipboard
Step logs removing API and Button
Closes #3444 closes #1272
Admin:
User:
go-sdk missing
Deployment of preview was torn down
also as per CI :D -> go generate cmd/server/swagger.go and commit
also as per CI :D -> go generate cmd/server/swagger.go and commit
Unfortunately, nothing changed :( go-generate-swagger.log
usr
before deleting (admin
also have access to this button):
usr2
after deleteion, the same for guest:
Can you allow this also for repo push access? The endpoint to delete the whole pipeline's logs is also available with push access only.
Ideally would be if the delete button would be hidden under the sub menu under the button with horizontal ...
icon or something like that (not requirement for this PR)
@6543 you requested changes, but your concerns were addressed @zc-devs Can you fix the linters?
Codecov Report
Attention: Patch coverage is 0%
with 39 lines
in your changes are missing coverage. Please review.
Project coverage is 35.92%. Comparing base (
8e45ddd
) to head (b9fc970
). Report is 4 commits behind head on main.
:exclamation: Current head b9fc970 differs from pull request most recent head de8e1d6. Consider uploading reports for the commit de8e1d6 to get more accurate results
Files | Patch % | Lines |
---|---|---|
server/api/pipeline.go | 0.00% | 39 Missing :warning: |
Additional details and impacted files
@@ Coverage Diff @@
## main #3451 +/- ##
==========================================
- Coverage 35.98% 35.92% -0.07%
==========================================
Files 231 231
Lines 15553 15602 +49
==========================================
+ Hits 5597 5605 +8
- Misses 9539 9580 +41
Partials 417 417
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@6543 can you take a look here? You requested changes so we cant merge without your approval
PS: sorry for the delay