Provides an option to remove the inessential version number from the generated models

[AndyButland]

Prerequisites

• I have added steps to test this contribution in the description below

Fixes: #15597

Description

The linked issue started with a point raised on why we include the commit hash in the version number. This does no harm to keep. But the discussion continued to ask if it's necessary to output the version number at all and established that it's not essential, but could be useful for debugging.

The downside - and we found this when working on umbraco.com - is every Umbraco upgrade leads to a lot of changes to these files, which are seemingly unnecessary and can clutter pull requests.

To avoid a behavioural breaking change, even if a harmless one, I've added configuration option to allow you to switch off the inclusion of the version number.

We could consider making this option default to false in a future version, but it's probably OK as is and can be turned on for people that want to avoid the version writing behaviour.

To Test:

• Configure models builder with source code manual:

"ModelsBuilder": {
  "ModelsMode": "SourceCodeManual",
}

• Create or edit a document type and verify changes are written to the output folder (by default at \umbraco\models\), and that they include the version number in the [GeneratedCode] attribute.
• Modify the configuration:

"ModelsBuilder": {
  "ModelsMode": "SourceCodeManual",
  "IncludeVersionNumberInGeneratedModels": false
}

• Create or edit a document type and verify changes are written to the output folder (by default at \umbraco\models\), and that they don't include the version number.

[mjpraxis]

@AndyButland is it the correct issue (#17998) this PR is fixing?

[AndyButland]

is it the correct issue (#17998) this PR is fixing?

No, sorry, I tagged the wrong one. Have updated now.