Skip to content

Conversation

@jbrockopp
Copy link
Contributor

Based off of #615 and #754

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

This adds a new user 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.

@jbrockopp jbrockopp added the enhancement Indicates an improvement to a feature label Apr 8, 2023
@jbrockopp jbrockopp self-assigned this Apr 8, 2023
@codecov
Copy link

codecov bot commented Apr 8, 2023

Codecov Report

Merging #809 (9bb2c47) into main (3335f6a) will increase coverage by 1.36%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #809      +/-   ##
==========================================
+ Coverage   56.83%   58.20%   +1.36%     
==========================================
  Files         243      242       -1     
  Lines       16099    15721     -378     
==========================================
  Hits         9150     9150              
+ Misses       6543     6165     -378     
  Partials      406      406              

@jbrockopp jbrockopp marked this pull request as ready for review April 8, 2023 01:34
@jbrockopp jbrockopp requested a review from a team as a code owner April 8, 2023 01:34
Copy link
Contributor

@ecrupper ecrupper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM — thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Indicates an improvement to a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants