Skip to content

Fix order of available projects in test_recipe#3023

Open
valeriupredoi wants to merge 2 commits intomainfrom
fix_test_recipe
Open

Fix order of available projects in test_recipe#3023
valeriupredoi wants to merge 2 commits intomainfrom
fix_test_recipe

Conversation

@valeriupredoi
Copy link
Contributor

@valeriupredoi valeriupredoi commented Mar 20, 2026

Description

Order got mangled for some reason - you know why @bouweandela ?


Before you get started

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.


To help with the number pull requests:

@bouweandela
Copy link
Member

Based on which tests are failing here: https://app.circleci.com/pipelines/github/ESMValGroup/ESMValCore?branch=main, it looks like something changed in a dependency. That test is a pain anyway, I've had to update it several times already. Maybe you could change it so it doesn't depend on the order of the projects and doesn't fail again when we add a another project?

@valeriupredoi
Copy link
Contributor Author

Based on which tests are failing here: https://app.circleci.com/pipelines/github/ESMValGroup/ESMValCore?branch=main, it looks like something changed in a dependency. That test is a pain anyway, I've had to update it several times already. Maybe you could change it so it doesn't depend on the order of the projects and doesn't fail again when we add a another project?

ah! Good call - I was suspecting it's a dependency, since we've not changed anything since yesterday 😁 Yes, let me have a closer look at it then

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants