Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

package.json: Is it LICENSE or LICENSE.txt or something else? #1178

Closed
pzrq opened this issue Mar 20, 2024 · 1 comment
Closed

package.json: Is it LICENSE or LICENSE.txt or something else? #1178

pzrq opened this issue Mar 20, 2024 · 1 comment

Comments

@pzrq
Copy link
Contributor

pzrq commented Mar 20, 2024

Expected Behaviour

The ... ui.frontend.../package.json file (ideally after generation) has a valid "license" key according to the npm documentation https://docs.npmjs.com/cli/v10/configuring-npm/package-json#license

For example, could it be one of the following?

  1. "license": "SEE LICENSE IN LICENSE",
  2. "license": "Apache-2.0", (from the npm-linked SPDX License List)
  3. Rename the license information file from LICENSE to LICENSE.txt

A rationale for this can be found, for example, when working with Amazon Inspector for SBOMs or similar legal compliance tooling at scale.

Actual Behaviour

This particular package.json file actually references a LICENSE.txt file which does not appear to exist:

"license": "SEE LICENSE IN LICENSE.txt",

Could it also (apart from a few brackets and whitespace) be more simply defined as Apache-2.0, like is done elsewhere?

Reproduce Scenario (including but not limited to)

Steps to Reproduce

I'm not sure exactly which command line my team used to generate the specific archetype for AEMaaCS which I saw this in, though absent other magic, I'd expect any archetype which contains a ui.frontend directory with a package.json file contained this issue.

Platform and Version

Most likely Windows 11, could also be macOS Sonoma or Ubuntu 22.04 LTS.

Sample Code that illustrates the problem

N/A

Logs taken while reproducing problem

N/A

@pzrq
Copy link
Contributor Author

pzrq commented Nov 27, 2024

Closing as I don't presently work on/with AEM, and due to inactivity.

@pzrq pzrq closed this as not planned Won't fix, can't repro, duplicate, stale Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant