Skip to content

Conversation

@ecrupper
Copy link
Contributor

Based off of #615 #754 #809 #828 #829 #847 #848 #849 #851 and #853

This change continues the refactor efforts initially introduced in the above PRs.

This adds a new log package to the github.com/go-vela/server/api package.

This contains all the same handlers that existed previously but with each of them within their own file.

Also changed GetBuildLogs to ListLogsForBuild to stay consistent.

@ecrupper ecrupper requested a review from a team as a code owner May 19, 2023 22:29
@ecrupper ecrupper self-assigned this May 19, 2023
@codecov
Copy link

codecov bot commented May 19, 2023

Codecov Report

Merging #855 (a15a60f) into main (1884ca0) will increase coverage by 1.73%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #855      +/-   ##
==========================================
+ Coverage   63.10%   64.84%   +1.73%     
==========================================
  Files         285      284       -1     
  Lines       14944    14543     -401     
==========================================
  Hits         9430     9430              
+ Misses       5068     4667     -401     
  Partials      446      446              

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.

4 participants