-
Notifications
You must be signed in to change notification settings - Fork 63
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
BUG: GPL License Violation? #1588
Comments
Hey @philipbjorge! Thank you so much for raising this topic and I appreciate that you've done some research already 😉
That's one of our top priorities as Bravado doesn't look to be maintained regularly for a while. Sadly there is no easy way to go without it and it will take us some time.
I've just checked that and it looks like all of our functional tests are passing without Based on all of the things above my suggestion is that you can uninstall this package on your side and there is probably no easy solution on our side to do that 😞 |
Hey @philipbjorge ,
Please let me know if this helps, or if you need further clarification 🙏 |
This is really helpful @SiddhantSadangi -- Thank you 👍 |
Describe the bug
We're trying to bring Neptune into our codebases and it's flagging some license violations for us.
Expected behavior
I would expect all the licenses in neptune's dependencies and transitive dependencies to be in compliance with your package's license.
I am not a lawyer, but I'm pretty sure the copyleft nature of GPL requires this package now to be GPL.
This is a downstream issue in
bravado-core
, but I wanted to raise awareness in case you had an easy path to exclude thebravado
dependency from this package.Yelp/bravado-core#261
Yelp/bravado-core#405
Workarounds
Untested, but I suspect we can fork bravado-core and remove the
format
extras in the requirements and use that.I'm not sure if Neptune depends on any of those functionalities.
Thanks
Appreciate you looking at this issue -- I know licensing and transitive dependencies aren't the most fun :)
The text was updated successfully, but these errors were encountered: