Skip to content

Add license metadata to package manifest#137

Open
martrend33 wants to merge 2 commits into
ether:mainfrom
martrend33:patch-1
Open

Add license metadata to package manifest#137
martrend33 wants to merge 2 commits into
ether:mainfrom
martrend33:patch-1

Conversation

@martrend33

Copy link
Copy Markdown

Summary
Adds Apache-2.0 license metadata to package.json, matching the repository LICENSE file and making the published npm metadata clearer.

Verification
Checked npm metadata for etherpad-cli-client 4.0.3 has no license field.
Confirmed repository LICENSE is Apache License 2.0.
Parsed the edited package.json and verified license is Apache-2.0.

Tests not run; metadata-only change.

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented Jun 11, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

PR Summary by Qodo

Add Apache-2.0 license field to package.json
⚙️ Configuration changes 🕐 Less than 5 minutes

Grey Divider

Walkthroughs

Description
• Add SPDX license metadata (Apache-2.0) to npm package manifest.
• Align published npm metadata with repository LICENSE file.
• Tidy package.json formatting/field placement around version/license.
High-Level Assessment

Using the standard npm "license" field with an SPDX identifier ("Apache-2.0") is the correct and most compatible approach for npm metadata. Alternatives (relying solely on the LICENSE file or using deprecated "licenses") provide worse registry visibility or compatibility.

Grey Divider

File Changes

Other (1)
package.json Add Apache-2.0 license metadata and tidy manifest layout +3/-1

Add Apache-2.0 license metadata and tidy manifest layout

• Adds a top-level "license": "Apache-2.0" field to the package manifest to ensure the npm registry displays correct licensing information. Also adjusts formatting/placement near the version field.

package.json


Grey Divider

Qodo Logo

@martrend33 martrend33 changed the title Add license metadata to package mAdd license metadata to package manifestAdd license metadata to package manifesAdd license metadata to package manifest Add license metadata to package manifest Jun 11, 2026
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

Successfully merging this pull request may close these issues.

1 participant