-
Notifications
You must be signed in to change notification settings - Fork 27
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
Licence Request #23
Comments
Hi MIT License The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software. THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. |
Hi Thanks @qiao for this great library! We have a similar issue as @tim-hitchins-ekkosense, we would like to make use of this library through another library. When you made a decision on which license you want to use, could you also update/include it on https://www.npmjs.com/package/heap? Currently it states license Thank you very much! |
I wanted to echo the concerns of Tim and Oliver above. PSF is not quite right here, could you apply MIT instead? One of my clients likes this library a lot and wants to use it, but the license is causing friction. Thanks for providing a great software library! |
@heathermeeker I'll note it's not even using the PSF. It's using the PSF but copied from Python 2.7.2:
That obviously makes no sense for this library, neither being written by the PSF or the software being Python 2.7.2. Clearly the user took the licence from the licence which applied to I think that means this software is unlicensed, and remains completely proprietary. That makes the 800k downloads a week a copyright + licence violation... 😢 |
Thanks for flagging. I've fixed the license in 21701c0 and published the new version to npm. |
@qiao Thank you so much! ❤️ |
Thank you so much! |
Hi,
We'd really like to use another project which depends on your library, but your library currently has no licence. This means we can't use either.
Given most of the packages on npm are MIT or Apache 2.0, we imagine you would choose one of these, but please could you clarify your position. If it would be helpful to you, we would be happy to open a pull request to add your choice of licence to the package for you.
Thanks,
Tim
The text was updated successfully, but these errors were encountered: