deps: Upgrade packages within constraints (tools/upgrade pub), mostly #873
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
Author
|
Updated to fix CI. The diff was just this: -// Autogenerated from Pigeon (v20.0.1), do not edit directly.
+// Autogenerated from Pigeon (v20.0.2), do not edit directly.at the top of the two Pigeon-generated files. |
Collaborator
|
Thanks! LGTM. While testing this out, some diff --git .gitignore .gitignore
index 4fee86d2a..a6ec1f646 100644
--- .gitignore
+++ .gitignore
@@ -5,9 +5,11 @@
*.swp
.DS_Store
.atom/
+.build/
.buildlog/
.history
.svn/
+.swiftpm/
migrate_working_dir/Not sure why. There isn't a |
Member
Author
|
Thanks! Merging.
That sounds like something the … Yep, looks like this upstream commit: So a quick PR to apply that would be good. |
This fixes zulip#825 because of the updates to file_picker 8.0.7 and flutter_local_notifications_linux 4.0.1. Holding back the Drift-related packages (drift, drift_dev, sqlparser) just because there appear to be some changes there that require small updates on our side, and today I'm catching up on things and so don't have time to dig into those myself. There are also some updates that `tools/upgrade pub-major` would make beyond these, which I'm similarly leaving aside for the moment. Fixes: zulip#825
chrisbobbe
added a commit
to chrisbobbe/zulip-flutter
that referenced
this pull request
Aug 22, 2024
This change has been getting applied by `flutter run`, and Greg identified this upstream commit as the cause: flutter/flutter@29f332c82 Discussion: zulip#873 (comment)
chrisbobbe
added a commit
that referenced
this pull request
Sep 3, 2024
This change has been getting applied by `flutter run`, and Greg identified this upstream commit as the cause: flutter/flutter@29f332c82 Discussion: #873 (comment)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Following up on #872 (comment) .
This fixes #825 because of the updates to file_picker 8.0.7 and
flutter_local_notifications_linux 4.0.1.
Holding back the Drift-related packages (drift, drift_dev, sqlparser)
just because there appear to be some changes there that require small
updates on our side, and today I'm catching up on things and so don't
have time to dig into those myself.
There are also some updates that
tools/upgrade pub-majorwouldmake beyond these, which I'm similarly leaving aside for the moment.
Fixes: #825