[FR]: Allow replacing the license text with SPDX License Identifier

[sarahec]

Contact Details

[email protected]

Plugin Version

1.0.12

What do you propose?

It would be useful to add the SPDX-License-Identifier below the copyright. The question I have is should we make this an option in the settings? I'm happy to submit a PR.

[minherz]

Hi,
I am not aware of a need to have both SPDX identifier and license text in the same file. The lint-like solutions either validate the presence of the license code with optional copyright verification or looking for SPDX identifier but not for both.
Can you elaborate a use case where having such identifier is required?

Regarding your question, if such an option will be added, it has to be an opt-in option.

[sarahec]

Ah, I misunderstood how the license identifier works. Thanks for clarifying.

[sarahec]

After discussion off-line, we've decided to make this an either-or switch: either it's the full license text or it's a copyright and license header. I"ll set up a PR.

[minherz]

I suggest to have this as a checkbox configuration that controls whether to add the full license text or the SPDX id only. Copyright info should be added anyway.