Skip to content

Update detsys-ts: Grahamc/switch to npm (#121)#231

Merged
grahamc merged 1 commit intoDeterminateSystems:mainfrom
detsys-pr-bot:detsys-ts-update-f335f0d0939f2b8488174d3cd5f818d1cec23ab6
Dec 10, 2025
Merged

Update detsys-ts: Grahamc/switch to npm (#121)#231
grahamc merged 1 commit intoDeterminateSystems:mainfrom
detsys-pr-bot:detsys-ts-update-f335f0d0939f2b8488174d3cd5f818d1cec23ab6

Conversation

@detsys-pr-bot
Copy link
Copy Markdown

@detsys-pr-bot detsys-pr-bot commented Dec 10, 2025 •
edited by coderabbitai bot
Loading

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

Summary by CodeRabbit

  • Chores
    • Updated development dependencies for code formatting improvements.

✏️ Tip: You can customize this high-level summary in your review settings.

@grahamc @github-actions
Update detsys-ts for: `Grahamc/switch to npm (DeterminateSystems#121)…
0470744 
…` (`f335f0d0939f2b8488174d3cd5f818d1cec23ab6`)
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Dec 10, 2025 •
edited
Loading

Walkthrough

The pull request increments the prettier devDependency from version ^3.7.3 to ^3.7.4 in package.json. This is a patch-level version update with no modifications to code logic, public APIs, or control flow.

Changes

Cohort / File(s) Summary
Dependency Update
package.json		 Bumped prettier devDependency from ^3.7.3 to ^3.7.4

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • grahamc

Poem

🐰 A dash, a dot, a version bump so neat,
Prettier grows more elegant and sweet,
From .3 to .4, the formatting flows,
Making code prettier wherever it goes! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title references 'detsys-ts' update and switching to npm, but the actual change only updates the 'prettier' devDependency from ^3.7.3 to ^3.7.4, which is unrelated to the title's claims. Update the title to accurately reflect the actual change, such as 'Update prettier devDependency to ^3.7.4' or align the PR changes with the stated objectives about detsys-ts and npm migration.
✅ 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.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 45146d6 and 0470744.

⛔ Files ignored due to path filters (2)
  • dist/index.js is excluded by !**/dist/**
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • package.json (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: typescript-action
🔇 Additional comments (1)
package.json (1)

42-42: Patch version update for dev dependency is safe.

Prettier 3.7.4 includes bug fixes for comments in union types and LWC interpolations, which are improvements over the previous patch. The caret constraint allows this patch update, and since prettier is a development-only dependency (not in production code), this update carries minimal risk.

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 2235257 into DeterminateSystems:main Dec 10, 2025
3 checks passed
This was referenced Jan 21, 2026
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@grahamc grahamc grahamc approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@detsys-pr-bot @grahamc