search

StefanMaron / vsc-lintercop

MIT License
6 stars 6 forks source link

Json Syntax for LinterCop.json #7

Closed jwikman closed 1 year ago

jwikman commented 1 year ago

Together with https://github.com/StefanMaron/BusinessCentral.LinterCop/pull/265 this is an attempt to solve https://github.com/StefanMaron/BusinessCentral.LinterCop/issues/216.

You would probably want to released this fix a while after the BusinessCentral.LinterCop.dll file is released, I believe.

I had to increase the dependency on vscode, to enable the "deprecated" property in the syntax file. But I believe that most developers are pretty good at keeping VSCode updated, so it should not be a major issue.

jwikman commented 1 year ago

Hi @StefanMaron,

Any thoughts on this PR?

jwikman commented 1 year ago

@StefanMaron I just noticed that you had to hotfix the dll due to me missing the enableRule0016ForApiObjects setting. I totally missed that one, my apologies!

The json syntax is now updated with enableRule0016ForApiObjects as well.

We are about to roll out LC on all customer repos, so it would be great if this got published soon. :)