StefanMaron / BusinessCentral.LinterCop

Community driven code linter for AL (MS Dynamics 365 Business Central)
https://stefanmaron.com
MIT License
77 stars 31 forks source link

LC0025 - Update Rule Title #484

Closed ans-bar-bm closed 9 months ago

ans-bar-bm commented 9 months ago

The current title / message Procedure must be either local or internal. at a quick glance feels a bit lacking or even confusing at times ("why should this procedure which is clearly referenced in that other extension be local?").

Updating the wording to something like Procedure must be either local, internal or define a documentation comment. might be a bit clearer on the intent of the rule (without having to read the detailed rule description in the wiki).

Arthurvdv commented 9 months ago

Thank you for reporting this. I now see that the current description can be confusing indeed. The documentation comment part is later added I believe, where the description hasn't been adapted to that change.

In the (pre)release version of v0.30.11 of the LinterCop the rule will have the description you've suggested.