Skip to content

Remove coln-ls license#70

Open
incipit0 wants to merge 3 commits into
mainfrom
coln-ls-rm-license
Open

Remove coln-ls license#70
incipit0 wants to merge 3 commits into
mainfrom
coln-ls-rm-license

Conversation

@incipit0

Copy link
Copy Markdown
Collaborator

I think I can remove a license since it is copyrighted by me?

@incipit0 incipit0 changed the title license: REMOVE Remove coln-ls license Jun 23, 2026
@github-actions

Copy link
Copy Markdown

Web demo preview: https://pr-70--coln-web-demo.netlify.app

@olynch

olynch commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Ah, I didn't notice this! You should remove this license file, and make sure that the copyright in the package.json or whatever is "Coln contributors" and then also that the license metadata in the package.json is Apache-2.0 OR MIT.

@incipit0 incipit0 force-pushed the coln-ls-rm-license branch from d5ad437 to 36d320e Compare June 24, 2026 12:51

@olynch olynch left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple nitpicks

"private": true,
"author": "Coln contributors",
"type": "module",
"license": "(MIT OR Apache-2.0)",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to be consistent, I've been doing Apache-2.0 OR MIT.

"LICENSE-APACHE"
],
"scripts": {
"copy-license": "cp ../../LICENSES/MIT.txt LICENSE-MIT && cp ../../LICENSES/Apache-2.0.txt LICENSE-APACHE",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to copy the license files? Also, maybe just cp -r ../../LICENSES LICENSES?

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.

2 participants