Skip to content

Update detsys-ts: Drop the old schemas and integrate the open PRs (#162)#108

Merged
grahamc merged 2 commits intoDeterminateSystems:mainfrom
detsys-pr-bot:detsys-ts-update-24d2aa4cffcb4fe9def110e054d79beda382e6e7
Mar 25, 2026
Merged

Update detsys-ts: Drop the old schemas and integrate the open PRs (#162)#108
grahamc merged 2 commits intoDeterminateSystems:mainfrom
detsys-pr-bot:detsys-ts-update-24d2aa4cffcb4fe9def110e054d79beda382e6e7

Conversation

@detsys-pr-bot
Copy link
Copy Markdown

@detsys-pr-bot detsys-pr-bot commented Mar 25, 2026

Bump detsys-ts to the latest HEAD and regenerate application bundle.

Summary by CodeRabbit

  • Chores
    • Updated development dependency version to the latest patch release.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 1ee46dc5-6fd3-483b-b586-112d66beefbf

📥 Commits

Reviewing files that changed from the base of the PR and between c12af21 and 526d81a.

⛔ Files ignored due to path filters (3)
  • dist/index.js is excluded by !**/dist/**
  • flake.lock is excluded by !**/*.lock
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • package.json

📝 Walkthrough

Walkthrough

This pull request updates the @types/node development dependency in package.json from version ^20.19.33 to ^20.19.37, incorporating the latest type definitions for Node.js 20.x.

Changes

Cohort / File(s) Summary
Development Dependencies
package.json
Bumped @types/node from ^20.19.33 to ^20.19.37 to incorporate the latest Node.js type definitions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • lucperkins
  • grahamc

Poem

🐰 Hop along with types so true,
Node's definitions fresh and new,
Twenty point nineteen takes its bound,
With patch thirty-seven, wisdom's found!
TypeScript knows, and we do too.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title describes dropping old schemas and integrating open PRs, but the actual changeset only updates a @types/node dev dependency version in package.json. The title should accurately reflect the primary changes in the changeset. Consider a more specific title like 'Update @types/node dependency to v20.19.37' or align the PR scope with the stated objectives.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@grahamc grahamc merged commit 87e91e1 into DeterminateSystems:main Mar 25, 2026
7 checks passed
MordragT pushed a commit to re-gmbh/flake-checker-action that referenced this pull request Mar 27, 2026
…e nix store version (DeterminateSystems#108)` (`c7303495f43d348cac78091ef434443b1ef22485`)
MordragT pushed a commit to re-gmbh/flake-checker-action that referenced this pull request Mar 27, 2026
…-update-c7303495f43d348cac78091ef434443b1ef22485

Update `detsys-ts`: Capture the version of Nix in addition to the nix store version (DeterminateSystems#108)
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.

2 participants