Skip to content

Add abvx-set#33628

Merged
matthewfeickert merged 3 commits into
conda-forge:mainfrom
markoblogo:markoblogo-abvx-set
Jun 13, 2026
Merged

Add abvx-set#33628
matthewfeickert merged 3 commits into
conda-forge:mainfrom
markoblogo:markoblogo-abvx-set

Conversation

@markoblogo

Copy link
Copy Markdown
Contributor

Checklist

  • Title of this PR is meaningful: e.g. "Adding my_nifty_package", not "updated meta.yaml".
  • License file is packaged.
  • Source is from official source.
  • Package does not vendor other packages.
  • If static libraries are linked in, the license of the static library is packaged.
  • Package does not ship static libraries.
  • Build number is 0.
  • A tarball (url) rather than a repo (e.g. git_url) is used in your recipe.
  • 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.

Notes

  • Package: abvx-set 0.2.4 from PyPI
  • Type: pure Python noarch: python
  • Smoke tests: module import plus set-validate-registry --help and set-plan-config-apply --help

@markoblogo

Copy link
Copy Markdown
Contributor Author

I am willing to be listed as a recipe maintainer for this feedstock as @markoblogo.

@markoblogo

Copy link
Copy Markdown
Contributor Author

@conda-forge-admin, please ping conda-forge/help-python

@conda-forge-webservices

Copy link
Copy Markdown

Hi! This is the friendly automated conda-forge-webservice.

I was asked to ping @conda-forge/help-python and so here I am doing that.

@conda-forge-admin

Copy link
Copy Markdown
Contributor

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/abvx-set/recipe.yaml) and found it was in an excellent condition.

@matthewfeickert matthewfeickert left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks pretty good. Just some minor revisions.

Comment thread recipes/abvx-set/recipe.yaml Outdated
build:
number: 0
noarch: python
script: python -m pip install . -vv --no-deps --no-build-isolation

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

c.f. https://conda-forge.org/docs/maintainer/example_recipes/pure-python/#the-build-section

Suggested change
script: python -m pip install . -vv --no-deps --no-build-isolation
script: ${{ PYTHON }} -m pip install . -vv --no-deps --no-build-isolation

Comment thread recipes/abvx-set/recipe.yaml Outdated
imports:
- scripts
pip_check: true
python_version: ${{ python_min }}.*

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

c.f. https://conda-forge.org/docs/maintainer/example_recipes/pure-python/#the-tests-section

Suggested change
python_version: ${{ python_min }}.*
python_version:
- ${{ python_min }}.*
- "*"

@matthewfeickert matthewfeickert left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@matthewfeickert matthewfeickert added this pull request to the merge queue Jun 13, 2026
Merged via the queue into conda-forge:main with commit 5fb7a83 Jun 13, 2026
7 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants