StefanMaron / BusinessCentral.LinterCop

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

LC0033 #683

Closed Arthurvdv closed 1 month ago

Arthurvdv commented 1 month ago

Discussed in https://github.com/StefanMaron/BusinessCentral.LinterCop/discussions/679

Originally posted by **SorenBogelund-ECIT** July 8, 2024 Thank you developing this great LinterCop! Why do I get a warning about the runtime version falling behind, when I haven't specified any runtime version in app.json? ![image](https://github.com/StefanMaron/BusinessCentral.LinterCop/assets/169046245/bfb6f39b-89f7-4ad7-96e8-4a274b7d7661)
Arthurvdv commented 1 month ago

This issue is related on some way to https://github.com/StefanMaron/BusinessCentral.LinterCop/issues/668.

While re-examining this rule, I see now the logic behind this has changed. In previous versions, not specifying an runtime property would return al null value and this diagnostic will not be raised. This now has been changed that it will return the CurrentRelease version, which points to 13.0 instead of 13.1 in the AL Language version 13.1.1065068. 🤨

Arthurvdv commented 1 month ago

I think I have a way of preventing this false positive. This should now be resolved in the (pre)release version of v0.30.28 of the LinterCop.

Arthurvdv commented 1 month ago

The version v0.30.28 of the LinterCop is now the latest release.