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

Stagger fortnightly scheduled uv/yarn updates #977

Merged
merged 1 commit into from
Nov 1, 2024

Conversation

ml-evs
Copy link
Member

@ml-evs ml-evs commented Oct 31, 2024

I think it makes sense to stagger the Python/JS ecosystem updates, so if we have any issues in the week after merging one, we can be fairly sure it was the cause. This PR staggers the updates so that Python updates on Wednesdays in the 1st and 3rd week of a month, and JS in the 2nd and 4th.

@ml-evs ml-evs added build For issues/PRs pertaining to the build or deployment of the package CI For issues/PRs regarding the continuous integration labels Oct 31, 2024
Copy link

codecov bot commented Oct 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.38%. Comparing base (dfd4cff) to head (a5e9820).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #977   +/-   ##
=======================================
  Coverage   68.38%   68.38%           
=======================================
  Files          62       62           
  Lines        3929     3929           
=======================================
  Hits         2687     2687           
  Misses       1242     1242           

Copy link

cypress bot commented Oct 31, 2024

datalab    Run #2682

Run Properties:  status check passed Passed #2682  •  git commit 26583db379 ℹ️: Merge a5e9820dcd447a8c4b216e3bc3ae001eb29a1451 into dfd4cff429e09d71177855fb2c99...
Project datalab
Branch Review ml-evs/ci-deps-schedule
Run status status check passed Passed #2682
Run duration 07m 07s
Commit git commit 26583db379 ℹ️: Merge a5e9820dcd447a8c4b216e3bc3ae001eb29a1451 into dfd4cff429e09d71177855fb2c99...
Committer Matthew Evans
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 402
View all changes introduced in this branch ↗︎

Copy link
Contributor

@BenjaminCharmes BenjaminCharmes left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@ml-evs ml-evs merged commit b38c99f into main Nov 1, 2024
17 checks passed
@ml-evs ml-evs deleted the ml-evs/ci-deps-schedule branch November 1, 2024 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build For issues/PRs pertaining to the build or deployment of the package CI For issues/PRs regarding the continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants