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
internal: Use lib crates from crates.io #15098
Conversation
This should help reduce breakages to the autopublish workflow. #15038 |
# In-tree crates that are published separately and follow semver. | ||
line-index = { version = "0.1.0-pre.1", path = "lib/line-index" } | ||
la-arena = { version = "0.3.1", path = "lib/la-arena" } | ||
lsp-server = { version = "0.7.1", path = "lib/lsp-server" } | ||
# In-tree crates that are published separately and follow semver. See lib/README.md | ||
line-index = { version = "0.1.0-pre.1" } | ||
la-arena = { version = "0.3.1" } | ||
lsp-server = { version = "0.7.1" } |
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.
Up above is a [patch.'crates-io']
section with a lot of commented out patches, could you add commented out path versions for these there? That would make it easier to use the local ones on demand for testing.
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.
Good idea. Done. I also amended the last bit that I'd added to libs/README.md accordingly.
See lib/README.md for justification
7bcd75d
to
5c8d1a4
Compare
Thanks |
☀️ Test successful - checks-actions |
See lib/README.md for justification