-
Notifications
You must be signed in to change notification settings - Fork 131
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 Turbo Cache to Native Link #402
Rename Turbo Cache to Native Link #402
Conversation
c6c5936
to
52e4d99
Compare
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.
Reviewed 6 of 237 files at r1.
Reviewable status: 6 of 238 files reviewed, 2 unresolved discussions (waiting on @allada and @MarcusSorealheis)
-- commits
line 2 at r1:
Nit: Remove trailing period.
README.md
line 3 at r1 (raw file):
# Native Link [![CI](https://github.com/trace_machina/native-link/workflows/CI/badge.svg)](https://github.com/trace_machina/native-link/actions/workflows/main.yml)
This (and probably a bunch of other places) is an invalid link. trace_machina
-> TraceMachina
.
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.
Reviewable status: 6 of 238 files reviewed, 2 unresolved discussions (waiting on @aaronmondal and @allada)
README.md
line 3 at r1 (raw file):
Previously, aaronmondal (Aaron Siddhartha Mondal) wrote…
This (and probably a bunch of other places) is an invalid link.
trace_machina
->TraceMachina
.
I think this was resolved just before you reviewed the PR, see: 52e4d99
52e4d99
to
d9c5b8e
Compare
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.
Reviewed 231 of 237 files at r1, 16 of 18 files at r2, 101 of 101 files at r3, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @allada and @MarcusSorealheis)
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.
Gonna immediately merge this so that we don't need additional giant rebases on this.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @allada and @MarcusSorealheis)
Description
This PR renames turbo-cache to be native link as this project has expanded well beyond build caching.
Fixes # (issue)
Type of change
rename
not work as expected)
How Has This Been Tested?
Please also list any relevant details for your test configuration
Checklist
bazel test //...
passes locallygit amend
see some docsThis change is![Reviewable](https://camo.githubusercontent.com/1541c4039185914e83657d3683ec25920c672c6c5c7ab4240ee7bff601adec0b/68747470733a2f2f72657669657761626c652e696f2f7265766965775f627574746f6e2e737667)