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

Add an upgrade section #6

Open
wants to merge 5 commits into
base: v03
Choose a base branch
from
Open

Add an upgrade section #6

wants to merge 5 commits into from

Conversation

mpizenberg
Copy link
Member

This is the first in a series of PRs that are intended to be merged into the v03 branch of this guide repo.
Once all v03 guide upgrades are done, we can merge it all into the main branch and deploy.

This PR adds a section describing the differences between v0.2 and v0.3 such that people understand what they have to update in their code.

@mpizenberg mpizenberg mentioned this pull request May 25, 2022
4 tasks
pub trait DependencyProvider<P: Package, V: Version> {
/// Compute the priority of a given package and range of versions for the solver.
/// When deciding which package to pick next, the solver will choose the highest priority one.
fn prioritize(&self, package, version_set) -> // priority

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It could help to include an example of the types that might be commonly used for Priority.

@Eh2406
Copy link
Member

Eh2406 commented Jan 30, 2025

@konstin, I kind of forgot about this PR in our ongoing work to release 0.3 (sorry @mpizenberg) is any of it still useful and helpful? Perhaps if not here in the change log?

@konstin
Copy link
Member

konstin commented Jan 30, 2025

yes we can recycle some of that for the changelog

@mpizenberg
Copy link
Member Author

Please take it over, I don’t have much time to work on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants