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

Consider another package name? lint? linter? source_lint? #57171

Closed
kevmoo opened this issue Feb 11, 2015 · 10 comments
Closed

Consider another package name? lint? linter? source_lint? #57171

kevmoo opened this issue Feb 11, 2015 · 10 comments
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.

Comments

@kevmoo
Copy link
Member

kevmoo commented Feb 11, 2015

dart_ is a bit redundant.

CC @sethladd for his perspective

@sethladd
Copy link
Contributor

Our friends in naming would probably like "Linter for Dart".

@pq
Copy link
Member

pq commented Feb 12, 2015

👍 for "Linter for Dart" for the human-readable/product name, but what about the package name? Is "linter" too open-ended?

@sethladd
Copy link
Contributor

For the record, we also have dart_style in our Org.

package name "linter" is fine, because it's scoped to pub.dartlang.org

Formal project name: "Linter for Dart" is great.

Repo name: linter, because it's scoped to dart-lang Org

@kevmoo
Copy link
Member Author

kevmoo commented Feb 12, 2015

+1!

On Thu, Feb 12, 2015 at 10:45 AM, Seth Ladd [email protected]
wrote:

For the record, we also have dart_style in our Org.

package name "linter" is fine, because it's scoped to pub.dartlang.org

Formal project name: "Linter for Dart" is great.

Repo name: linter, because it's scoped to dart-lang Org


Reply to this email directly or view it on GitHub
https://github.com/dart-lang/dart_lint/issues/25#issuecomment-74127398.

@pq
Copy link
Member

pq commented Feb 12, 2015

Works for me. @sethladd : can you do the move? I'm guessing/hoping history, issues, etc will make the transition?

@seaneagan
Copy link

Is this package going to expose a command-line app? If so, I find it nice to have the name of the command-line app match that of the package, and 'linter' wouldn't be a great name for a command-line app. Just something to consider.

@pq
Copy link
Member

pq commented Feb 12, 2015

Absolutely. I've got a provisional commandline entry point already. Thanks for chiming in @seaneagan!

@sethladd
Copy link
Contributor

Go ahead and rename this repo. GitHub is very good at redirects.

and yes, matching command-line names is generally a good idea.

@seaneagan
Copy link

Besides a command-line app, another place where namespacing with dart or dart_ would come in handy be #57153. The configuration file should be namespaced e.g. .dartlintrc instead of .linterrc, and it's much easier to mentally connect that to a dartlint or dart_lint package than a linter package.

@pq
Copy link
Member

pq commented Feb 12, 2015

Rename done. That said, @seaneagan, you bring up some good points. Let's take the config file conversation over to #57153...

@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.
Projects
None yet
Development

No branches or pull requests

5 participants