search

microsoft / AL

Home of the Dynamics 365 Business Central AL Language extension for Visual Studio Code. Used to track issues regarding the latest version of the AL compiler and developer tools available in the Visual Studio Code Marketplace or as part of the AL Developer Preview builds for Dynamics 365 Business Central.
MIT License
744 stars 245 forks source link

Modify Rule AA0215 #6307

Closed IceOnly closed 3 years ago

IceOnly commented 3 years ago

Based on the Best Pratice Guide (https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/compliance/apptest-bestpracticesforalcode#file-naming), the suffix/prefix in the filename is not optional.

The open issue for the guide is not comment since three months (https://github.com/MicrosoftDocs/dynamics365smb-devitpro-pb/issues/1361)

Guide or rule should be modified to match each other.

I would prefer to revert the guide to the old definition:

\<ObjectNameExcludingPrefix>.al

dzzzb commented 3 years ago

It seems the documentation was updated to match the actual practice of VS Code, so isn't this resolved?

https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/compliance/apptest-bestpracticesforalcode#file-naming

In your extension, the name of each new application object (table, page, codeunit), can contain a prefix or suffix.

The CodeCop analyzer suggests that the object name is part of the file name, which is encouraged as a best practice. Adding any affixes to the file names is voluntary.

IceOnly commented 3 years ago

Yes, issue solved!