Skip to content

Fix the license mismatch and fill in the package.json metadata - #9

Merged
Megapixel99 merged 2 commits into
masterfrom
fix/license-mismatch-mit
Aug 21, 2026
Merged

Megapixel99 merged 2 commits into
masterfrom
fix/license-mismatch-mit

Conversation

@Megapixel99

@Megapixel99 Megapixel99 commented Aug 21, 2026 •

Copy link
Copy Markdown
Owner

package.json still carried three npm init defaults: "license": "ISC", "description": "", and "author": "".

The license one was an actual inconsistency — LICENSE.md at the repo root holds the MIT text and the README's License section says MIT, so the manifest contradicted both. It now says "license": "MIT". No license text changes.

The other two are filled in to match what the repo already says about itself:

  • description — the README's opening line.
  • author — Seth Wheeler <23089578+Megapixel99@users.noreply.github.com>, the identity on all 42 commits in the repo.

On renaming LICENSE.md to LICENSE

Considered and skipped. GitHub already resolves the current filename — gh repo view Megapixel99/nodejs-k8s --json licenseInfo returns {"key":"mit","name":"MIT License"}, so the sidebar picks it up as-is. Renaming would only invalidate the README's see \LICENSE.md`` reference for no gain.

Context: found while preparing a submission to sindresorhus/awesome-nodejs, where a clearly stated license and populated metadata are worth having.

🤖 Generated with Claude Code

`package.json` carried `"license": "ISC"`, the `npm init` default, while
LICENSE.md holds the MIT text and the README says MIT. MIT is the intent,
so the manifest now says so.
Both were still the `npm init` defaults. The description follows the
README's opening line; the author matches the identity on every commit in
the repo.
@Megapixel99 Megapixel99 changed the title Declare MIT in package.json to match LICENSE.md Fix the license mismatch and fill in the package.json metadata Aug 21, 2026
@Megapixel99
Megapixel99 force-pushed the fix/license-mismatch-mit branch from 5ea6859 to bf0467b Compare August 21, 2026 14:48
@Megapixel99
Megapixel99 merged commit 85fe606 into master Aug 21, 2026
4 checks passed
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