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

Request: use more sensible defaults #290

Open
DeflateAwning opened this issue Mar 23, 2024 · 4 comments
Open

Request: use more sensible defaults #290

DeflateAwning opened this issue Mar 23, 2024 · 4 comments

Comments

@DeflateAwning
Copy link

I'm getting both of the following errors when I switch the style of my module name back and forth between ExampleModule and exampleModule:

Begin `module` name with lowerCamelCase.svls(lowercamelcase_module)
Begin `module` name with UpperCamelCase.svls(uppercamelcase_module)

I don't really care which one it is, but one should be selected and should be used as the default. I can't contend with it wanting both simultaneously.

I'm sure there are countless other examples of this sort of too-configurable-to-be-usable with the default setup.

@thomasdingemanse
Copy link
Contributor

Would it be an idea to use one of the presets as the default unless a .svls.toml is found in the filesystem hierarchy? To me that would be a more sensible default than enabling everything regardless of conflicts. I'd be happy to submit a PR if this seems like a good idea.

@DeflateAwning
Copy link
Author

I think this sounds like an obvious improvement!

@thomasdingemanse
Copy link
Contributor

It just occurred to me that it probably makes more sense to implement this in the svlint repository.

@DeflateAwning
Copy link
Author

Potentially makes sense to do in both

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

No branches or pull requests

2 participants