Skip to content

make sure parse_recipe_yaml is called with cbc_path#5610

Merged
beckermr merged 1 commit intoregro:mainfrom
minrk:always-parse-v1-cbc
Feb 21, 2026
Merged

make sure parse_recipe_yaml is called with cbc_path#5610
beckermr merged 1 commit intoregro:mainfrom
minrk:always-parse-v1-cbc

Conversation

@minrk
Copy link
Contributor

@minrk minrk commented Feb 21, 2026

Description:

it's frequently required to parse recipes (v1 at least) with cbc config, either from the recipe or .ci_support.

I only found one ci_support record in feedstock_attrs, so use that. I'm not sure if there's a better value to use or a better way to find the key? I could only find "the first one that starts with ci_support".

Fixes, for example,

create_migration_yaml_creator(migrators, gx, pin_to_debug="mpich")

Checklist:

  • Pydantic model updated or no update needed

Cross-refs, links to issues, etc:

closes #5607

it's ~always required to parse recipes with cbc config, either from the recipe or .ci_support

I only found one ci_support record in feedstock_attrs, so use that
@minrk minrk closed this Feb 21, 2026
@minrk minrk reopened this Feb 21, 2026
@codecov
Copy link

codecov bot commented Feb 21, 2026

Codecov Report

❌ Patch coverage is 94.11765% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 79.78%. Comparing base (6fc17ea) to head (960bdf6).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
conda_forge_tick/make_migrators.py 94.11% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5610      +/-   ##
==========================================
+ Coverage   79.74%   79.78%   +0.03%     
==========================================
  Files         144      144              
  Lines       17006    17018      +12     
==========================================
+ Hits        13562    13577      +15     
+ Misses       3444     3441       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@beckermr beckermr merged commit 84e95b8 into regro:main Feb 21, 2026
18 of 20 checks passed
@minrk minrk deleted the always-parse-v1-cbc branch February 21, 2026 16:53
@minrk
Copy link
Contributor Author

minrk commented Feb 21, 2026

This appeared to fix the mpich bot PRs: conda-forge/conda-forge-pinning-feedstock#8269

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

make_migrator doesn't load variables from conda_build_config.yaml

2 participants