-
Notifications
You must be signed in to change notification settings - Fork 22
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
Add an explicit license. #17
Comments
I hope this request for smoothing over the meta-context of the repository can bypass the
|
Yeah, that's a reasonable request. I don't have full context here, but I recall when the repo was created that they weren't ready to choose a license and the guy most involved is currently out on paternity leave until end of January. Since this isn't urgent, I'll wait til he's back and then circle back with him. |
In the meantime I've left this attribution where I've copied the tests to describing the licensing situation; https://github.com/Skenvy/dependabot-linguist/blob/af2b44f1c6747aec9c4129a65f9e47a19b4fc940/smoke-test/README.md which I hope is fine. |
Yeah, that's totally fine for now. I don't know, but I suspect the internal question is just whether we need the prosperity license from If for some reason we have to move to a license that doesn't allow copying, we can always reach out to you down the road. But in the meantime please don't let this block what you're working on as long as it's work on a free OSS project. |
Now that I think that we might need to add an explicit |
Ping, this would be timely. No license at all on a GitHub repo is not a great practice. |
@Jeffrey-Luszcz ping: is this something you may be able to help resolve? |
At the moment, the closest to this is the README saying
But it would be nice if the licensing was less vague and ephemeral.
I ask because I'd like to duplicate the files here in a set of tests to run github's linguist on. Linguist can't run on submodules, and I'd like for the unit tests to not have to rely on an external repo, or multiple versions of it given the pinned versions that the smoke tests here guarentee to be runnable with dependabot.
The text was updated successfully, but these errors were encountered: