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

Refactor based on flutter_lints #8

Merged
merged 3 commits into from
Apr 23, 2022

Conversation

myConsciousness
Copy link
Contributor

@myConsciousness myConsciousness commented Apr 23, 2022

Hi amazing developer,

This library is already a great completion, but I have noticed that some of the coding does not follow Dart or Flutter standards because it does not use flutter_lints.

Therefore, I have made the following fixes.

  • Added flutter_lints to pubspec.yaml
  • Added analysis_options.yaml
  • Fixed some lines based on flutter_lints suggestions (Not destructive)
  • Renamed files based on flutter_lints suggestions (Not destructive)

Thank you.

@myConsciousness myConsciousness changed the title Refactor based on flutter_lint Refactor based on flutter_lints Apr 23, 2022
@myConsciousness
Copy link
Contributor Author

@Jamalianpour

Let me know if you have any questions about the changes :)

@Jamalianpour
Copy link
Owner

Hello @myConsciousness, and thank you very much.

@Jamalianpour Jamalianpour merged commit 5243431 into Jamalianpour:master Apr 23, 2022
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.

2 participants