Skip to content

Update README.md #858

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 30, 2021
Merged

Update README.md #858

merged 1 commit into from
Jul 30, 2021

Conversation

domesticmouse
Copy link
Contributor

@domesticmouse domesticmouse commented Jul 28, 2021 •
edited
Loading

Hey @gaaclarke, using a flutter hash as the required version is a tad baroque. Can we just say Flutter v2 is the minimum?

@domesticmouse domesticmouse requested a review from gaaclarke July 28, 2021 06:28
@google-cla google-cla bot added the cla: yes Contributor license agreement signed by all authors label Jul 28, 2021
redbrogdon
redbrogdon approved these changes Jul 29, 2021
View reviewed changes
Copy link
Contributor

@redbrogdon redbrogdon left a comment

Choose a reason for hiding this comment

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

I think we can probably just run with this.

@gaaclarke
Copy link
Member

Sorry, just seeing this. I think that's correct, v2. Thanks.

@domesticmouse domesticmouse merged commit 89093ca into master Jul 30, 2021
@domesticmouse domesticmouse deleted the domesticmouse-add-to-app branch July 30, 2021 00:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@redbrogdon redbrogdon redbrogdon approved these changes

@gaaclarke gaaclarke Awaiting requested review from gaaclarke

Assignees
No one assigned
Labels
cla: yes Contributor license agreement signed by all authors
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@domesticmouse @gaaclarke @redbrogdon