-
Notifications
You must be signed in to change notification settings - Fork 16
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
rename project to Kargo Render #185
Conversation
Signed-off-by: Kent <[email protected]>
✅ Deploy Preview for docs-bookkeeper-akuity-io ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #185 +/- ##
=======================================
Coverage 38.28% 38.28%
=======================================
Files 19 19
Lines 1157 1157
=======================================
Hits 443 443
Misses 697 697
Partials 17 17
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couldn't find any leftover bookkeeper references - LGTM!
Holding for additional LGTM from @jessesuen |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Fixes #181
This PR renames the project to Kargo Render.
The new configuration file (previously
Bookfile.yaml
) iskargo-render.yaml
.New package name is
render
, so some exported types likeRenderRequest
andRenderResponse
got renamed to avoid the so-called "stutter" ofrender.RenderRequest
, etc.There are no functional changes in this PR.
Some non-code follow-up work may be required -- corresponding changes to CodeCov, Netlify, etc.