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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add package:effective_dart #1781

Merged
merged 2 commits into from
Oct 22, 2019
Merged

Add package:effective_dart #1781

merged 2 commits into from
Oct 22, 2019

Conversation

tenhobi
Copy link
Contributor

@tenhobi tenhobi commented Oct 21, 2019

Hello,

I have updated the effective dart lint rules I previously added to this repository and replaced it by the packages:effective_dart, which covers those updates and can be included in projects in way as package:pedantic can be.

This will make updating the list easier and we can make versions etc.

Thanks,
let me know if there is any trouble. 馃憤

@kevmoo kevmoo requested a review from pq October 21, 2019 20:48
Copy link
Member

@pq pq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

馃憤

Thanks!

@tenhobi
Copy link
Contributor Author

tenhobi commented Oct 22, 2019

Travis fails for some reason tho. It seems it drop maintenance to 90 from 100 for some reason?

@pq
Copy link
Member

pq commented Oct 22, 2019

I'm not able to get anything from travis at all... Looks like they're possibly having infrastructure issues?

Can you paste the end of the failed build here and I can look at it? (Assuming you have it...)

@pq
Copy link
Member

pq commented Oct 22, 2019

https://travis-ci.org/dart-lang/linter/builds/601269524?utm_source=github_status&utm_medium=notification

@tenhobi
Copy link
Contributor Author

tenhobi commented Oct 22, 2019

@pq
image

image

@pq
Copy link
Member

pq commented Oct 22, 2019

Hmmm. Running the baseline locally fails for me too which (w/o your change) as it looks like we've got a dependency to upgrade. I'll have a fix for that in a sec.

FWIW, here's what we're running:

dart tool/pana_baseline.dart

(The output is not great.)

The smoking gun for your build is this bit:

  "suggestions": [
   {
    "code": "pubspec.dependencies.outdated",
    "level": "warning",
    "title": "Support latest dependencies.",
    "description": "The version constraint in `pubspec.yaml` does not support the latest published versions for 1 dependency (`analyzer`).",
    "score": 10.0
   }
  ]
 },

@pq
Copy link
Member

pq commented Oct 22, 2019

OK, so this comes down to a new available analyzer package; unfortunately, there does not appear to be a compatible dart_style package available so we're stuck at the moment.

The good news that the travis infrastructure issues seem fixed so at least I can see the failed build!

I'll go ahead and merge and pickup the dependency issue separately.

Thanks for the contribution!

@tenhobi
Copy link
Contributor Author

tenhobi commented Oct 22, 2019

OK, thanks. :-)

@tenhobi
Copy link
Contributor Author

tenhobi commented Oct 24, 2019

@pq I will ask here, since this PR was merged a while ago.

Should I, and can I, make badges to the linter web page as well as flutter and pedantic does? (http://dart-lang.github.io/linter/lints/)

I think that would be cool feature.

@pq
Copy link
Member

pq commented Oct 24, 2019

Great idea @tenhobi!

@tenhobi
Copy link
Contributor Author

tenhobi commented Oct 24, 2019

I will make separate PR tomorrow, thanks! 馃挋

@pq
Copy link
Member

pq commented Oct 24, 2019

Super. Let me know if you hit any snags!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants