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

App template generates MIT License file without prompt #1079

Closed
2 tasks
kjdelisle opened this issue Mar 2, 2018 · 11 comments
Closed
2 tasks

App template generates MIT License file without prompt #1079

kjdelisle opened this issue Mar 2, 2018 · 11 comments
Assignees
Milestone

Comments

@kjdelisle
Copy link
Contributor

kjdelisle commented Mar 2, 2018

Overview

Generating a new application with the lb4 app command generates a LICENSE file, whose contents are the MIT license template. This happens without user interaction!

Problems

  • Users can't select a different license
  • Users who don't pay attention may end up releasing code to others under a license they never intended (not something we should make possible, however unlikely)
  • Users can't opt-out of selecting a license of any kind

@strongloop/loopback-devs What are your thoughts on this?

Acceptance Criteria

As a result:

  • In package.json, have a license property set to empty string ("").
    • This ensures npm install will give a warning to users about the project not having a license.
  • Not generating LICENSE file at all
@dhmlau
Copy link
Member

dhmlau commented Mar 3, 2018

That's good catch. I didn't realize LICENSE file got generated!
IMHO, i don't think we should generate the LICENSE file for the user.

@virkt25
Copy link
Contributor

virkt25 commented Mar 5, 2018

+1 on not providing a license file at all

@shimks
Copy link
Contributor

shimks commented Mar 5, 2018

+1

1 similar comment
@jannyHou
Copy link
Contributor

jannyHou commented Mar 5, 2018

+1

@raymondfeng
Copy link
Contributor

There are debates on how npm init handles license. See npm/npm#8918 and https://spdx.org/spdx-specification-21-web-version.

I suggest that we generate the following:

  1. Add "license": "SEE LICENSE IN LICENSE" to package.json
  2. Generate an empty LICENSE file

@dhmlau
Copy link
Member

dhmlau commented Mar 5, 2018

@raymondfeng , any reasons that we want to generate an empty LICENSE file vs not generating it at all?
Either way we go, I'd like to mark this as MVP.

@raymondfeng
Copy link
Contributor

@dhmlau

  1. It's a good practice to include the license with any Node.js module/app
  2. npm checks SPDX compliance for license in package.json. You will see warnings during npm i if license property is missing or not complying to SPDX.

@b-admike
Copy link
Contributor

b-admike commented Mar 5, 2018

+1 for keeping it out and I like the idea of generating an empty LICENSE file. It won't hurt if it doesn't end up getting used.

@virkt25
Copy link
Contributor

virkt25 commented Mar 6, 2018

Rejecting. Needs more clarification. Acceptance Criteria needs to address problems. Concerns / questions raised during estimation below.


  • What does LoopBack 3 do?
  • What are the implications of an empty License file vs. not having a license field at all?
  • We can't shield everyone across the globe from all the legal implications. We aren't in the business of LICENSE compliance.
  • Users should take ownership
  • Should CLI tell the user to add a LICENSE themselves
  • What does a npm expect for a published package?

@dhmlau
Copy link
Member

dhmlau commented Mar 9, 2018

What does LoopBack 3 do?

No LICENSE file.
In package.json, "license": "UNLICENSED",
(I've tried with LB3 creating an "empty server" app)

What are the implications of an empty License file vs. not having a license field at all?
We can't shield everyone across the globe from all the legal implications. We aren't in the business of LICENSE compliance.

Questions for @jjtang1

Users should take ownership

Yes

Should CLI tell the user to add a LICENSE themselves

It wouldn't hurt. but again, question for @jjtang1

What does a npm expect for a published package?

See this link in npm documentation.
From @raymondfeng's above comment, we'll get a warning for not specifying the license attribute.

@dhmlau
Copy link
Member

dhmlau commented Mar 12, 2018

@raymondfeng @strongloop/sq-lb-apex @bajtos , are you ok with following what we do in LB3:
i.e.

  • In package.json, use license": "UNLICENSED"
  • Not genenerating LICENSE file at all

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants