Support recipe v1 in Build2HostMigrator#3808
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3808 +/- ##
==========================================
- Coverage 77.73% 76.29% -1.45%
==========================================
Files 136 136
Lines 14779 14787 +8
==========================================
- Hits 11489 11282 -207
- Misses 3290 3505 +215 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
beckermr
left a comment
There was a problem hiding this comment.
A few things but otherwise looks great! Thank you!
isuruf
left a comment
There was a problem hiding this comment.
These migrators are for easy transition from conda-build 2 to 3. I think we should drop this mini-migrator.
|
Well, either way is fine with me. I've figured out it makes sense to keep it, in case people |
|
I agree it can be dropped. For that, we need to remove it from the various bits of the make_migrators file. Once we do that we can merge. |
Description:
Add support for v1 recipes in
Build2HostMigrator, and extend the relevant tests. I've also replaced the explicit schema version check with asuper()call.Checklist:
Cross-refs, links to issues, etc:
From the list on #3642