Skip to content

CI job for updating flake.lock#146

Open
Xendergo wants to merge 1 commit into
masterfrom
push-nvmyytuxkkrm
Open

CI job for updating flake.lock#146
Xendergo wants to merge 1 commit into
masterfrom
push-nvmyytuxkkrm

Conversation

@Xendergo

@Xendergo Xendergo commented Jun 14, 2026

Copy link
Copy Markdown
Collaborator

If building the webring fails due to the flake.lock falling out of date, the deploy script will attempt to update the flake and see whether that fixes the build issue. If so, it will push the updated flake.lock to master.

This also makes the nix build use the pinned rust version in rust-toolchain.toml. Using the pinned rust version wouldn't work without the ability to automatically update the lockfile.

@codecov

codecov Bot commented Jun 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.86%. Comparing base (5b87efd) to head (971a566).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #146   +/-   ##
=======================================
  Coverage   87.86%   87.86%           
=======================================
  Files           8        8           
  Lines        2430     2430           
  Branches     2430     2430           
=======================================
  Hits         2135     2135           
  Misses        261      261           
  Partials       34       34           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Jun 14, 2026

Copy link
Copy Markdown

Test Results

46 tests   46 ✅  2s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit 971a566.

♻️ This comment has been updated with latest results.

@Xendergo Xendergo force-pushed the push-nvmyytuxkkrm branch 2 times, most recently from 48f859d to 021795e Compare June 14, 2026 17:43
@Xendergo Xendergo marked this pull request as ready for review June 14, 2026 17:46
@Xendergo Xendergo force-pushed the push-nvmyytuxkkrm branch from 021795e to 9c4713c Compare June 14, 2026 17:48
ericswpark
ericswpark previously approved these changes Jun 14, 2026

@ericswpark ericswpark left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, would be nice if the check interval is shorter but that's just IMHO

Comment thread .github/workflows/update-flake.yml Outdated
@Xendergo Xendergo marked this pull request as draft June 14, 2026 18:56
@Xendergo Xendergo force-pushed the push-nvmyytuxkkrm branch 2 times, most recently from 5cd20a0 to 608ffca Compare June 15, 2026 01:52
@Xendergo Xendergo marked this pull request as ready for review June 15, 2026 01:52
@Xendergo Xendergo requested a review from ericswpark June 15, 2026 01:53
kdkasad
kdkasad previously approved these changes Jun 15, 2026
Comment thread .github/workflows/ci.yml Outdated
Comment thread ci/update-flake-lock.sh
ericswpark
ericswpark previously approved these changes Jun 15, 2026
@Xendergo Xendergo dismissed stale reviews from ericswpark and kdkasad via d65ce68 June 15, 2026 02:22
@Xendergo Xendergo force-pushed the push-nvmyytuxkkrm branch 3 times, most recently from b1dd138 to 2bfce6a Compare June 15, 2026 02:23
@Xendergo Xendergo force-pushed the push-nvmyytuxkkrm branch from 2bfce6a to 971a566 Compare June 15, 2026 02:31
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