Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove scheduled pixi auto-update and add scheduled build with latest dependencies #191

Merged
merged 2 commits into from
Apr 23, 2024

Conversation

traversaro
Copy link
Member

This is a bit of iteration on #178 and what is described in prefix-dev/pixi#910 .

The idea of #178 is that once in a while we wanted to update the pixi lock file, to ensure that the software does not stop working with new versions of the dependencies. However, periodic update of the pixi lock file have the downside that the repo size continue to grow, even if the repo is not changed at all. Furthermore, the PR for the update for the lock needs to be manually merged, and they add noise in the PRs for the repo.

After a bit of experience with the solution added in #178, I thought of another solution: added a scheduled job that deletes the pixi.lock and run the CI on a freshly created pixi.lock, to test the compilation and tests with the latest version of the dependencies (as we used to do with the conda-ci). In this way, we frequently test the latest versions, without changing the pixi.lock, unless necessary. Note that I left the pixi-auto-update-ci action, as it can be useful to update the pixi.lock via workflow_dispatch via the GitHub Actions. The frequency of the scheduled job is bi-weekly.

Co-authored-by: Pavel Zwerschke <pavelzw@gmail.com>
@traversaro traversaro merged commit cef18fa into master Apr 23, 2024
9 checks passed
@traversaro traversaro deleted the changepixiupdatestrategy branch April 23, 2024 12:43
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.

3 participants