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

Flutter 3.3 lint fixes #681

Merged
merged 4 commits into from
Nov 3, 2022

Conversation

diegotori
Copy link
Collaborator

@diegotori diegotori commented Nov 3, 2022

Library and Example App are now using flutter_lints.

Addresses any outstanding Dart Analysis issues as a result. Mostly just renaming variables.

@diegotori
Copy link
Collaborator Author

@brianegan and/or @maherjaafar, could use a quick once-over on these changes, since they are minor at best. Mostly CI YAML cleanup and standardizing lints across the board.

Thanks in advance.

@diegotori
Copy link
Collaborator Author

Otherwise, I'll merge it myself in the morning.

@diegotori
Copy link
Collaborator Author

Self-approving, since this merely affects static analysis.

@diegotori diegotori merged commit 9261f8b into fluttercommunity:master Nov 3, 2022
diegotori added a commit that referenced this pull request Nov 25, 2022
lg8294 added a commit to lg8294/chewie that referenced this pull request Jan 3, 2023
* master:
  Version 1.3.6. Addresses PRs fluttercommunity#676 and fluttercommunity#681.
  fix default value for minimum safe area
  Cleaned up the Github Workflows YAML.
  Fixed remaining Dart Analysis issues after switching to flutter_lints.
  Example app is now using flutter_lints.
  Library is now using flutter_lints.
  Changed documentation as requested
  Use EdgeInsets.zero
  Allow Chewie controls to be positioned to allow for a larger safe area.
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.

1 participant