Skip to content

Lint for outdated pubspec dependency constraints #57182

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

Closed
seaneagan opened this issue Feb 18, 2015 · 2 comments
Closed

Lint for outdated pubspec dependency constraints #57182

seaneagan opened this issue Feb 18, 2015 · 2 comments
Labels
devexp-linter Issues with the analyzer's support for the linter package legacy-area-analyzer Use area-devexp instead. linter-lint-request type-enhancement A request for a change that isn't a bug

Comments

@seaneagan
Copy link

One should CONSIDER keeping your dependencies up-to-date to:

  • get bug fixes.
  • avoid dependency resolution conflicts with other packages that have updated.
  • use new features and stop using deprecated features.

This could be a separate tool, e.g. see https://david-dm.org/, but it would be quite similar in purpose and implementation to the other pub lints e.g. #57175. For example, many pub lints would be reported on a YamlNode from the pubspec. And a quick fix would edit that node.

den already has functionality to detect and fix outdated dependencies via its CLI, and a dart API will be available soon.

@pq
Copy link
Member

pq commented Feb 18, 2015

@seaneagan: I love the idea of a user notification that deps are out of date. Playing with it as a lint seems reasonable to me. (And certainly expedient.) If it makes sense elsewhere we can certainly migrate.

Keep me posted on the dart API. In the meantime, I'm most of the way done with the basic parser/visitor bits to plumb in pubspec lints. I'll ping you when it lands and we can start playing.

Cheers!

@pq
Copy link
Member

pq commented Jun 8, 2020

I think this is now more or less handled by the pub outdated command and any improved UX should probably tie it into something like an IntelliJ "inspection" (rather than a lint).

/fyi @jwren @alexander-doroshko

@pq pq closed this as completed Jun 8, 2020
@devoncarew devoncarew added devexp-linter Issues with the analyzer's support for the linter package legacy-area-analyzer Use area-devexp instead. labels Nov 18, 2024
@devoncarew devoncarew transferred this issue from dart-archive/linter Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devexp-linter Issues with the analyzer's support for the linter package legacy-area-analyzer Use area-devexp instead. linter-lint-request type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

3 participants