Skip to content

pubspec: detect two identical keys #57187

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

Open
guillermooo opened this issue Mar 2, 2015 · 2 comments
Open

pubspec: detect two identical keys #57187

guillermooo opened this issue Mar 2, 2015 · 2 comments
Labels
area-devexp For issues related to the analysis server, IDE support, linter, `dart fix`, and diagnostic messages. devexp-linter Issues with the analyzer's support for the linter package linter-lint-request P3 A lower priority bug or feature request type-enhancement A request for a change that isn't a bug

Comments

@guillermooo
Copy link
Contributor

I think pubspec-related linting is going to be handled by linter. If so, it could detect cases like this:

name: dart_sample_console
version: 0.0.1
description: A minimal command-line application.
environment:
  sdk: '>=1.0.0 <2.0.0'
dependencies:
  semver: '>=1.0.0 <2.0.0'
  semver: '>=0.1.0 <1.0.0'
dev_dependencies:
  unittest: any

However, I'm not sure how linter is intended to run (on demand, constantly in the background...). In this particular case, it would be useful if the user got instant feedback, as the analysis server provides for .dart files.

@guillermooo
Copy link
Contributor Author

Moved from: dart-lang/dart_style#191

@pq
Copy link
Member

pq commented Mar 2, 2015

Regarding how the linter is meant to run, there will be a few options. One of them will be interactively via the server. That's a little ways off but not too far.

As for the request, this is awesome. Keep 'em coming! :)

@pq pq added type-enhancement A request for a change that isn't a bug linter-lint-request labels Mar 2, 2015
@srawlins srawlins added the P3 A lower priority bug or feature request label May 22, 2023
@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
@bwilkerson bwilkerson added area-devexp For issues related to the analysis server, IDE support, linter, `dart fix`, and diagnostic messages. and removed legacy-area-analyzer Use area-devexp instead. labels Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-devexp For issues related to the analysis server, IDE support, linter, `dart fix`, and diagnostic messages. devexp-linter Issues with the analyzer's support for the linter package linter-lint-request P3 A lower priority bug or feature request type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

5 participants