Skip to content

Conversation

@jaimergp
Copy link
Member

PR Checklist:

  • note any issues closed by this PR with closing keywords
  • if you are adding a new page under docs/ or community/, you have added it to the sidebar in the corresponding _sidebar.json file
  • put any other relevant information below

Closes Quansight-Labs/conda-ecosystem-sta-mgmt#63

@jaimergp jaimergp requested a review from a team as a code owner November 21, 2025 16:09
@netlify
Copy link

netlify bot commented Nov 21, 2025

Deploy Preview for conda-forge-previews ready!

Name Link
🔨 Latest commit efa7f2c
🔍 Latest deploy log https://app.netlify.com/projects/conda-forge-previews/deploys/6932b82cfbd58200086cb4e1
😎 Deploy Preview https://deploy-preview-2661--conda-forge-previews.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 78
Accessibility: 96
Best Practices: 100
SEO: 89
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Open your `conda-forge.yml` file and add these lines:

```yaml
abi_migration_branches: # TODO
Copy link
Contributor

Choose a reason for hiding this comment

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

It this a TODO for the doc or meant literally?

Copy link
Member Author

Choose a reason for hiding this comment

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

Oops, nop, I missed this. Added in 9d27f98

Copy link
Contributor

@mgorny mgorny left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@h-vetinari h-vetinari left a comment

Choose a reason for hiding this comment

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

This LGTM, though I think we may want to call out the following behaviour that can be annoying/surprising:

Creating a branch off of an old commit and adding it to abi_migration_branches will almost certainly cause the bot to create a whole bunch of pointless migrator PRs against that new branch, because the logic for when to create such a migrator PR is very sensitive to even the most minor rerender changes, which will almost always trigger for old commits (i.e. stuff that hasn't been rerendered in a while).

I still think it would be good to avoid this by double-rerendering for the maintenance branches, see regro/cf-scripts#2500

@jaimergp
Copy link
Member Author

jaimergp commented Dec 2, 2025

Good to know. I'll add a quick note but perhaps we should focus on fixing that surprising behavior too.

@jaimergp jaimergp merged commit 303f59f into conda-forge:main Dec 5, 2025
6 checks passed
@jaimergp jaimergp deleted the how-to-versions branch December 5, 2025 11:05
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.

Diátaxis: How-to guides - Advanced - Maintain several versions

3 participants