Skip to content

Conversation

pvditto
Copy link
Contributor

@pvditto pvditto commented Oct 9, 2025

Also includes a minor workflow fix.

The test failures are CI issues AFAICT.

The workflow was using flutter-version: "stable" which caused the action
to interpret "stable" as both the version and channel, resulting in the
error "Unable to determine Flutter version for channel: stable version: stable".

Fixed by using flutter-version: '3.x' with channel: 'stable', matching
the pattern used in the flutter-ci.yml workflow.
@pvditto pvditto self-assigned this Oct 9, 2025
@pvditto pvditto marked this pull request as ready for review October 10, 2025 18:27
@pvditto pvditto requested a review from phatblat as a code owner October 10, 2025 18:27
@Copilot Copilot AI review requested due to automatic review settings October 10, 2025 18:27
Copilot

This comment was marked as outdated.

Copy link
Contributor

@konstantinbe konstantinbe left a comment

Choose a reason for hiding this comment

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

Swift LGTM.

@pvditto
Copy link
Contributor Author

pvditto commented Oct 13, 2025

Reverted changes that I had included for testing and forgot to remove again.

@pvditto pvditto requested a review from Copilot October 13, 2025 10:08
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 17 out of 30 changed files in this pull request and generated no new comments.

Files not reviewed (2)
  • javascript-tui/package-lock.json: Language not supported
  • javascript-web/package-lock.json: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@cameron1024
Copy link
Contributor

Hello, it's me from beyond the grave 👋 💀

I think I may need to be removed from codeowners or something (or maybe just unassigned from this PR? not sure exactly - but I am still in codeowners).

Anyways Flutter changes LGTM 😢 🫡

Copy link
Contributor

@teodorciuraru teodorciuraru left a comment

Choose a reason for hiding this comment

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

Approved. The CI fixes are in a PR that waits to get merged into main. It's ok to jump over them for now if changes seem stable.

@phatblat phatblat removed the request for review from cameron1024 October 15, 2025 14:58
@pvditto pvditto merged commit 2e0f546 into main Oct 16, 2025
66 of 72 checks passed
@pvditto pvditto deleted the pv/bump-sdk-4.12.3 branch October 16, 2025 15:08
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.

7 participants