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

Complaints about this library declaration which seems valid to me. #57179

Closed
zoechi opened this issue Feb 18, 2015 · 4 comments
Closed

Complaints about this library declaration which seems valid to me. #57179

zoechi opened this issue Feb 18, 2015 · 4 comments
Assignees
Labels
analyzer-linter Issues with the analyzer's support for the linter package area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)

Comments

@zoechi
Copy link
Contributor

zoechi commented Feb 18, 2015

library bwu_server.shared.datastore.some_file;

DO name libraries and source files using lowercase_with_underscores. (.../my_package/lib/src/shared/datastore/some_file.dart, line 1, col 1)

Are you interested in such bug reports already?

@pq pq added the type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) label Feb 18, 2015
@pq pq self-assigned this Feb 18, 2015
@pq
Copy link
Member

pq commented Feb 18, 2015

Cool. Absolutely yes, @zoechi, bug reports and feature requests most welcome.

Thanks!

(Lint in question: #57164.)

@pq
Copy link
Member

pq commented Feb 20, 2015

Should be fixed. Please file any other issues you catch.

Thanks!

@zoechi
Copy link
Contributor Author

zoechi commented Feb 20, 2015

Great! It would be even more fun to test/use if there were any way to disable checks like linter --ignore=3 17 25

@pq
Copy link
Member

pq commented Feb 20, 2015

Agreed! I think we need to hammer this out sooner than later. Feel free to chime in over on #57153 if you have thoughts.

Again, thanks for the report. Keep 'em coming!

@devoncarew devoncarew added analyzer-linter Issues with the analyzer's support for the linter package area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. labels Nov 18, 2024
@devoncarew devoncarew transferred this issue from dart-lang/linter Nov 18, 2024
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analyzer-linter Issues with the analyzer's support for the linter package area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Projects
None yet
Development

No branches or pull requests

3 participants