Skip to content
This repository has been archived by the owner on Jun 13, 2024. It is now read-only.

migrate from ui.hash* to Object.hash* #654

Merged
merged 3 commits into from
Apr 25, 2022
Merged

migrate from ui.hash* to Object.hash* #654

merged 3 commits into from
Apr 25, 2022

Conversation

werainkhatri
Copy link
Member

@werainkhatri werainkhatri commented Apr 24, 2022 •
edited
Loading

Part fix for flutter/flutter#85431

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read the Flutter Style Guide recently, and have followed its advice.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

guidezpl
guidezpl approved these changes Apr 25, 2022
View reviewed changes
Copy link
Member

@guidezpl guidezpl left a comment

Choose a reason for hiding this comment

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

Thanks! I'm looking at fixing CI, then will merge this

@guidezpl guidezpl merged commit 63c749f into flutter:master Apr 25, 2022
@werainkhatri werainkhatri deleted the migrate-hash branch April 25, 2022 15:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@guidezpl guidezpl guidezpl approved these changes

@dnfield dnfield Awaiting requested review from dnfield

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@werainkhatri @guidezpl