-
Notifications
You must be signed in to change notification settings - Fork 558
Document josh subtrees and update subtree repository list #2182
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
Conversation
src/external-repos.md
Outdated
|
|
||
| `subtree` dependencies are currently managed by two distinct approaches: | ||
|
|
||
| * Using `git subtree` directly |
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.
| * Using `git subtree` directly | |
| * Using `git subtree` |
There's no "indirect" usage so I find the additional qualifier confusing
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.
I wanted to distinguish "using subtrees via josh" and "using subtrees via git subtree", but the word was extraneous, I suppose. Removed it.
src/external-repos.md
Outdated
| ## External Dependencies (subtrees) | ||
|
|
||
| As a developer to this repository, you don't have to treat the following external projects | ||
| differently from other crates that are directly in this repo: |
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.
That's not quite true, tools generally prefer PRs to their own repos when changes don't have to be done in lockstep with Rust compiler changes.
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.
This was pre-existing, but I agree it was confusing, I reworded it.
jieyouxu
left a comment
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.
Thanks!
|
|
||
| `subtree` dependencies are currently managed by two distinct approaches: | ||
|
|
||
| * Using `git subtree` |
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.
portable-simd also uses this; it seems to be entirely missing from this list?
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.
The list was incomplete before, I added a few entries, but since there was no global list before that, I didn't notice portable-simd. Good catch! Added it in #2214.
Mention our usage of josh in
miriandrust-analyzer, and add the cranelift backend to the list.