-
-
Notifications
You must be signed in to change notification settings - Fork 165
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
tighten_depends metis dependency in all packages that depends on metis #527
Conversation
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 ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a comment on what the patch is fixing.
Done, thanks. |
Once you are ready to merge this, make it not a draft a pr and we can go ahead! |
Sure, I am waiting on conda-forge/metis-feedstock#39 and then I will update the timestamp. |
e164210
to
5d76b46
Compare
(in a gist as it is too big for a comment) |
The PR is ready for review. |
Hi! This is the friendly conda-forge automerge bot! I considered the following status checks when analyzing this PR:
Thus the PR was passing and merged! Have a great day! |
As of 2023/08/29, two version of metis have been packaged on conda-forge:
5.1.0
5.1.1
Both version contain a
run_exports: {{ pin_subpackage('metis', max_pin='x.x') }}
. However, it emerged that there is an ABI breakage between 5.1.0 and 5.1.1 (see KarypisLab/METIS#71 (comment)), so we need to:max_pin='x.x.x'
(see Use stricter run_exports and fix version in headers for 5.1.1 metis-feedstock#38 and Create new metis 5.1.0 build with fixed run_exports metis-feedstock#39)The PR is currently in draft, as I will update it once conda-forge/metis-feedstock#38 is merged.
Checklist
generate_patch_json.py
if absolutely necessary.python show_diff.py
and posted the output as part of the PR.