Update to dulwich 1.0.0#10701
Conversation
From the release notes: > From here on, Dulwich will not break backwards compatibility until 2.0 jelmer/dulwich@ca9de6d
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdate the dulwich dependency constraint to allow version 1.x (up to but not including 2.0) and refresh the lockfile accordingly. File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- Given Dulwich’s backward-compatibility guarantee only starts at 1.0.0, consider bumping the lower bound to
>=1.0.0if you intend to rely on stable APIs while expanding the upper bound to<2.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Given Dulwich’s backward-compatibility guarantee only starts at 1.0.0, consider bumping the lower bound to `>=1.0.0` if you intend to rely on stable APIs while expanding the upper bound to `<2`.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Happy to do that if the poetry maintainers want to. But it works with the previous version as well. |
|
I think it is fine to only bump the upper bound. Apparently, there is no breaking change that hits us between 0.25 and 1.0 and I do not expect more 0.x versions in the future. |
Indeed, there will not be any more 0.x versions with one caveat: if there are security issues or grave C Git interoperability issues (new release of C Git write repositories in a way that breaks Dulwich) I will consider releasing another 0.25.x - but also without making any backwards incompatible changes. |
From the release notes: > From here on, Dulwich will not break backwards compatibility until 2.0
|
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
From the release notes:
jelmer/dulwich@ca9de6d
Pull Request Check List
Summary by Sourcery
Build: