Skip to content

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

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)#238
grahamc merged 2 commits intoDeterminateSystems:mainfrom
detsys-pr-bot:detsys-ts-update-24d2aa4cffcb4fe9def110e054d79beda382e6e7

Conversation

@detsys-pr-bot
Copy link
Contributor

@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 dependencies to latest compatible versions.

@coderabbitai
Copy link

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: 5bcacec9-1bc9-4c02-84c6-30d5d9b36f53

📥 Commits

Reviewing files that changed from the base of the PR and between 9a59e15 and 4f27303.

⛔ 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

Updated the @types/node development dependency from version ^20.19.33 to ^20.19.37 in package.json, applying a minor patch version bump to the TypeScript type definitions for Node.js.

Changes

Cohort / File(s) Summary
Dependency Update
package.json
Bumped @types/node from ^20.19.33 to ^20.19.37 in devDependencies.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • PR #228: Bumps the same @types/node devDependency further, continuing a sequential version progression.
  • PR #223: Updates @types/node devDependency to a different target version in the same codebase.
  • PR #203: Modifies the @types/node dependency entry in package.json with a version bump.

Suggested reviewers

  • grahamc
  • lucperkins

Poem

🐰 A tiny hop through type definitions,
Where Node's declarations find fruition,
From thirty-three to thirty-seven we go,
Patch by patch, our types will grow!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions 'Update detsys-ts' and 'Drop the old schemas', but the actual changeset only updates @types/node dependency in package.json. The title does not accurately reflect the primary change. Update the title to reflect the actual change, such as 'Update @types/node dependency to ^20.19.37' or clarify what detsys-ts changes are included in this PR.
✅ 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 enabled auto-merge (squash) March 25, 2026 00:34
@grahamc grahamc merged commit e02dcf8 into DeterminateSystems:main Mar 25, 2026
13 checks passed
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