adding acryl-datahub

[anshbansal]

Checklist

• Title of this PR is meaningful: e.g. "Adding my_nifty_package", not "updated meta.yaml".
• License file is packaged (see here for an example).
• Source is from official source.
• Package does not vendor other packages. (If a package uses the source of another package, they should be separate packages or the licenses of all packages need to be packaged).
• If static libraries are linked in, the license of the static library is packaged.
• Package does not ship static libraries. If static libraries are needed, follow CFEP-18.
• Build number is 0.
• A tarball (url) rather than a repo (e.g. git_url) is used in your recipe (see here for more details).
• GitHub users listed in the maintainer section have posted a comment confirming they are willing to be listed there.
• When in trouble, please check our knowledge base documentation before pinging a team.

[github-actions]

Hi! This is the staged-recipes linter and your PR looks excellent! 🚀

[conda-forge-admin]

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 (recipes/acryl-datahub/meta.yaml) and found it was in an excellent condition.

[anshbansal]

Commenting that I am ok being listed as maintainer

[pedro93]

Commenting that I am ok being listed as maintainer

[conda-forge-admin]

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 (recipes/acryl-datahub/meta.yaml) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipes/acryl-datahub/meta.yaml:

• ℹ️ noarch: python recipes should usually follow the syntax in our documentation for specifying the Python version.
  • For the host section of recipe, you should usually use python {{ python_min }} for the python entry.
  • If the package requires a newer Python version than the currently supported minimum version on conda-forge, you can override the python_min variable by adding a Jinja2 set statement at the top of your recipe (or using an equivalent context variable for v1 recipes).

_{This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12930350019. Examine the logs at this URL for more detail.}

[conda-forge-admin]

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 (recipes/acryl-datahub/meta.yaml) and found it was in an excellent condition.

[conda-forge-admin]

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 (recipes/acryl-datahub/meta.yaml) and found some lint.

Here's what I've got...

For recipes/acryl-datahub/meta.yaml:

• ❌ Non noarch packages should have python requirement without any version constraints.

_{This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12932584167. Examine the logs at this URL for more detail.}

[conda-forge-admin]

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 (recipes/acryl-datahub/meta.yaml) and found it was in an excellent condition.

[anshbansal]

@conda-forge/help-python, ready for review!

[hsheth2]

Commenting that I am ok being listed as maintainer

[synapticarbors]

This looks like this package should probably be noarch: python. See:
https://conda-forge.org/docs/maintainer/knowledge_base/#noarch-python

Take note of changes related to python_min. I think the main thing you were struggling with is that even though upstream says it supports python>=3.8, in reality that metadata is wrong since they are using typing syntax that would require you to set python_min = 3.10

[anshbansal]

There are tests running that check that it works on Python 3.8 https://github.com/datahub-project/datahub/actions/runs/12946781664

In the conda pipelines it was running it on Python 3.12

[synapticarbors]

It's not clear to me why this won't run on osx given the metadata in the upstream setup.py. In general though, I think this can be noarch: python and this skip section can be removed.

[anshbansal]

Removed win, osx and just keeping python skips