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

controllers/krate/metadata: Add support for default_version include mode #10288

Merged
merged 1 commit into from
Dec 31, 2024

Conversation

eth3lbert
Copy link
Contributor

This allows us to respond with a version data of default_version included, which potentially benefits apps by eliminating the need to wait for the crate response to obtain the default_version and then make a subsequent request to get the actual version data. This is particularly useful when we move towards not requesting with versions included.

@eth3lbert eth3lbert force-pushed the include-default-version branch from aa1833a to 6c191f9 Compare December 29, 2024 11:36
@eth3lbert eth3lbert force-pushed the include-default-version branch 3 times, most recently from 1b1be41 to 5e2da2c Compare December 30, 2024 10:43
@eth3lbert eth3lbert force-pushed the include-default-version branch from 5e2da2c to c27c857 Compare December 30, 2024 13:36
… mode

This allows us to respond with a version data of `default_version`
included, which potentially benefits apps by eliminating the need to
wait for the crate response to obtain the default version and then make
a subsequent request to get the actual version data. This is
particularly useful when we move towards not requesting with `versions`
included.
@eth3lbert eth3lbert force-pushed the include-default-version branch from c27c857 to 60142dc Compare December 30, 2024 13:49
@Turbo87 Turbo87 merged commit 9b89b10 into rust-lang:main Dec 31, 2024
8 checks passed
@eth3lbert eth3lbert deleted the include-default-version branch December 31, 2024 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants