-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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 PyPI package parquet-tools #15926
Conversation
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/parquet-tools:
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
I manually added a file with the MIT license. Should I also add a file with the Apache license? |
Hi @jdblischak thanks for submitting the recipe. Could you open an issue https://github.com/ktrueda/parquet-tools and ask about the License status? In the meantime maybe we should have it dual-licensed? Thoughts @conda-forge/staged-recipes ? On another note, the CI is showing the following error: |
|
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/parquet-tools:
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
A quick update on how I handled the licensing situation. Fortunately the maintainer was receptive and helpful. I sent PR ktrueda/parquet-tools#16 to
The maintainer published these changes on PyPI, and then I updated the feedstock in conda-forge/parquet-tools-feedstock#1 |
The license situation is a bit of a mess. I'd appreciate advice on how to to address these issues.
pyproject.toml
file specifies the package is MIT licensed, but there is no actual license file included in the repositoryparquet.thrift
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).