Skip to content
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

Convert _packet urls to z-pages #130

Merged
merged 1 commit into from
Nov 2, 2022

Conversation

chrisdoherty4
Copy link
Member

@chrisdoherty4 chrisdoherty4 commented Oct 25, 2022

Change /_packet endpoints to z-pages.

Closes #133

Signed-off-by: Chris Doherty <[email protected]>
@codecov
Copy link

codecov bot commented Oct 29, 2022

Codecov Report

Merging #130 (c59707f) into main (cff71d4) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #130   +/-   ##
=======================================
  Coverage   24.26%   24.26%           
=======================================
  Files           7        7           
  Lines         680      680           
=======================================
  Hits          165      165           
  Misses        493      493           
  Partials       22       22           
Impacted Files Coverage Δ
http/server.go 77.77% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jacobweinstock jacobweinstock added the ready-to-merge Signal to Mergify to merge the PR. label Nov 2, 2022
@chrisdoherty4
Copy link
Member Author

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Nov 2, 2022

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 3c54ee7

@mergify mergify bot merged commit 3c54ee7 into tinkerbell:main Nov 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Signal to Mergify to merge the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove vendor specific endpoints
2 participants