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

Update flutter_lints to 2.x #185

Closed
wants to merge 1 commit into from

Conversation

thedalelakes
Copy link

No description provided.

@cedvdb
Copy link

cedvdb commented May 26, 2022

were there 0 lint issues ? Usually just changing the number is not enough, lint issues will appear in the project

@thedalelakes
Copy link
Author

I did not run the update against the example code or the library. Does the update create new lint warnings?

@cedvdb
Copy link

cedvdb commented Jun 4, 2022

I don't know, I'm not the one submitting the PR. I'm asking if you checked ? If you use vs code it automatically shows up in the "problems" tab once you update the lint dep

@cedvdb
Copy link

cedvdb commented Jun 29, 2022

@DarshanGowda0

@jacksoncurrie
Copy link

jacksoncurrie commented Jul 2, 2022

Wouldn't it be a better idea to move the flutter_lints package to dev_dependencies: as it's only required for development? This would fix conflicts for every version going forward for anyone who uses the package.

See #190 for changes.

@thedalelakes
Copy link
Author

Agree w/ #190

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.

None yet

3 participants