Skip to content

python v3.13.2#772

Merged
conda-forge-admin merged 5 commits intoconda-forge:mainfrom
h-vetinari:bump
Feb 14, 2025
Merged

python v3.13.2#772
conda-forge-admin merged 5 commits intoconda-forge:mainfrom
h-vetinari:bump

Conversation

@h-vetinari
Copy link
Copy Markdown
Member

Fresh release: https://discuss.python.org/t/python-3-13-2-and-3-12-9-now-available/79509/2

Despite rebasing patch 0024, I'm not enabling it yet, as this is being addressed in #770

@conda-forge-admin
Copy link
Copy Markdown
Contributor

conda-forge-admin commented Feb 4, 2025

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

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

For recipe/meta.yaml:

  • ℹ️ The recipe is not parsable by parser conda-souschef (grayskull). This parser is not currently used by conda-forge, but may be in the future. We are collecting information to see which recipes are compatible with grayskull.
  • ℹ️ The recipe is not parsable by parser conda-recipe-manager. The recipe can only be automatically migrated to the new v1 format if it is parseable by conda-recipe-manager.

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

@h-vetinari
Copy link
Copy Markdown
Member Author

Ping @conda-forge/python for this and #773

Comment thread recipe/conda_build_config.yaml Outdated
Comment on lines +17 to +18
is_python_min:
- no
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.

Could we put this next to the other python keys from zip_keys? That would make it clearer this is related

python:
- 3.13
python_impl:
- cpython
numpy:
- 1.16

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Feel free to push into this PR if you feel strongly about it (same for #773).

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.

I know it sounds cosmetic, but there is a lot going on in this recipe (and this file). It helps a lot to understand the configurations here (especially for new contributors)

@jakirkham
Copy link
Copy Markdown
Member

Thanks Axel! 🙏

Had one suggestion above

@jakirkham
Copy link
Copy Markdown
Member

@conda-forge-admin , please re-render

1 similar comment
@jakirkham
Copy link
Copy Markdown
Member

@conda-forge-admin , please re-render

@conda-forge-admin
Copy link
Copy Markdown
Contributor

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

I tried to rerender for you, but it looks like there was nothing to do.

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

@isuruf isuruf added the automerge Merge the PR when CI passes label Feb 14, 2025
@conda-forge-admin conda-forge-admin merged commit ff62f86 into conda-forge:main Feb 14, 2025
@conda-forge-admin
Copy link
Copy Markdown
Contributor

Hi! This is the friendly conda-forge automerge bot!

I considered the following status checks when analyzing this PR:

  • linter: passed
  • azure: passed

Thus the PR was passing and merged! Have a great day!

@h-vetinari h-vetinari deleted the bump branch February 14, 2025 08:02
@jakirkham jakirkham mentioned this pull request Feb 14, 2025
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Merge the PR when CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants