Skip to content

Conversation

mmc1718
Copy link
Contributor

@mmc1718 mmc1718 commented Jul 20, 2025

Description

Fix for #11200

  • In MatchSpec.conda_build_form: Default to using * as the value for version parameter in the case where the value is not set (Falsey) in MatchSpec but build parameter is set
  • Add unit test for MatchSpec.conda_build_form

Checklist - did you ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update necessary tests?
  • Add / update outdated documentation?

@mmc1718 mmc1718 requested a review from a team as a code owner July 20, 2025 14:49
@github-project-automation github-project-automation bot moved this to 🆕 New in 🔎 Review Jul 20, 2025
@conda-bot
Copy link
Contributor

We require contributors to sign our Contributor License Agreement and we don't have one on file for @mmc1718.

In order for us to review and merge your code, please e-sign the Contributor License Agreement PDF. We then need to manually verify your signature, merge the PR (conda/infrastructure#1176), and ping the bot to refresh the PR.

@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Jul 22, 2025
travishathaway
travishathaway previously approved these changes Jul 22, 2025
Copy link
Contributor

@travishathaway travishathaway left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you for adding a test and a news item! 👏

@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Approved in 🔎 Review Jul 22, 2025
@travishathaway
Copy link
Contributor

@jaimergp,

Because you were the original author for this issue, could you please review it too?

@travishathaway travishathaway moved this from ✅ Approved to 👀 In Review in 🔎 Review Jul 22, 2025
Copy link

codspeed-hq bot commented Jul 22, 2025

CodSpeed Instrumentation Performance Report

Merging #15025 will not alter performance

Comparing mmc1718:set-default-version-conda-build-form (b129db9) with main (e7b992c)

Summary

✅ 21 untouched benchmarks

@jaimergp
Copy link
Contributor

Just a couple nits here and there, LGTM otherwise! Thanks!

Remove redundant assert statement, update punctuation in docs and news

Co-authored-by: jaimergp <jaimergp@users.noreply.github.com>
jezdez
jezdez previously approved these changes Jul 23, 2025
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Approved in 🔎 Review Jul 23, 2025
@travishathaway travishathaway enabled auto-merge (squash) July 30, 2025 08:40
@travishathaway
Copy link
Contributor

I had to resolve a merge conflict. The branch should merge once the tests are finished running 🚀

@travishathaway travishathaway merged commit 6f14a3d into conda:main Jul 30, 2025
75 checks passed
@github-project-automation github-project-automation bot moved this from ✅ Approved to 🏁 Done in 🔎 Review Jul 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed [bot] added once the contributor has signed the CLA
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants