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

bumped minimum Dart SDK constraint to 2.17 #356

Merged
merged 1 commit into from
Oct 24, 2022

Conversation

MaikuB
Copy link
Contributor

@MaikuB MaikuB commented Oct 23, 2022

0.7.4 declares that it depends xml 6.1.0+, which requires a minimum Dart SDK version of 2.17 but dbus still says the minimum it needs is 2.15. This PR is to address this so that it properly declares the minimum supported Dart SDK

Two things that came to mind when I noticed this though

  1. Potentially dbus should've published 0.7.4 as 0.8.0
  2. The build actions aren't catching this issue. I'm not that familiar with GitHub but from the workflows I've seen, the containers being referenced point to dart:stable. I suspect this could lead to a version drift of the sorts and the repository may need some way to check that the package still supports the minimum Dart SDK version that it advertises

@github-actions
Copy link

Hey! MaikuB has not signed the Canonical CLA which is required to get this contribution merged on this project.

Please head over to https://ubuntu.com/legal/contributors to read more about it.

@MaikuB
Copy link
Contributor Author

MaikuB commented Oct 23, 2022

Signed

@MaikuB
Copy link
Contributor Author

MaikuB commented Oct 23, 2022

@robert-ancell I can't seem to assign a reviewer and from what I know, you're the main person working on this so wanted to tag you to bring awareness in case this gets missed :)

@robert-ancell
Copy link
Contributor

Analyze issue fixed in #357

@robert-ancell
Copy link
Contributor

Thanks @MaikuB!

@robert-ancell robert-ancell merged commit f7dbc76 into canonical:main Oct 24, 2022
@MaikuB MaikuB deleted the minDartSdk branch October 25, 2022 09:58
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.

2 participants