search

dalance / svls

SystemVerilog language server
MIT License
466 stars 31 forks source link

Request: use more sensible defaults #290

Open DeflateAwning opened 8 months ago

DeflateAwning commented 8 months ago

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 commented 7 months ago

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 commented 7 months ago

I think this sounds like an obvious improvement!

thomasdingemanse commented 7 months ago

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

DeflateAwning commented 7 months ago

Potentially makes sense to do in both