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

replaced the deprecated pedantic package with the new lints package #305

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

danger-ahead
Copy link

Details

pedantic v1.1 is deprecated. lints is the new recommendation from the dart team. (https://pub.dev/documentation/pedantic/latest/)
Used the recommended lint options with the new lints package.

Pull Request Process

  1. Ensure any install or build dependencies are removed before the end of the layer when doing a build.
  2. Update the README.md if needed with details of changes to the interface, this includes new environment variables, exposed ports, useful file locations and container parameters.
  3. Increase the version numbers in any examples files and the README.md to the new version that this Pull Request would represent. The versioning scheme we use is SemVer.
  4. You may merge the Pull Request in once you have the sign-off of one developer, or if you do not have permission to do that, you may request the second reviewer to merge it for you.

@danger-ahead
Copy link
Author

@aagarwal1012 Please review the changes.

@weeklies
Copy link

Since this is a Flutter package, how about using https://pub.dev/packages/flutter_lints?

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