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

[Discussion] Use lints via package:lints #227

Open
devkabiir opened this issue Nov 16, 2019 · 2 comments
Open

[Discussion] Use lints via package:lints #227

devkabiir opened this issue Nov 16, 2019 · 2 comments

Comments

@devkabiir
Copy link
Contributor

devkabiir commented Nov 16, 2019

I believe that we do want to write effective dart code, but this might not be feasible for new learners. Sometimes too many lints can distracting. So I suggest that we use package:lints for our repo, this will cover all of the exercise examples and tests without necessarily forcing learners to use theses lints. We could then manually add exercise directories in check_formatting script like so dartanalyzer.bat --options=.\analysis_options.yaml .\exercises\anagram\lib .\exercises\anagram\test.

@md-weber
Copy link

That sound like a good plan, in my opinion. We could also consider the package:pedanitic

@Stargator
Copy link
Contributor

Stargator commented Oct 16, 2021

@md-weber Just want to note as of Jun 2021 the pedantic package has been discontinued

@devkabiir And the effective_dart package has been deprecated as well.

It seems now the the recommendation is to use the package: lints. I'll update the above ticket with that correction.

@Stargator Stargator changed the title [Discussion] Use package:effective_dart lints [Discussion] Use lints via package:lints Oct 16, 2021
@Stargator Stargator self-assigned this Oct 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants