Skip to content

Update detsys-ts: Capture the version of Nix in addition to the nix store version (#108)#211

Merged
grahamc merged 1 commit intoDeterminateSystems:mainfrom
detsys-pr-bot:detsys-ts-update-c7303495f43d348cac78091ef434443b1ef22485
Sep 3, 2025
Merged

Update detsys-ts: Capture the version of Nix in addition to the nix store version (#108)#211
grahamc merged 1 commit intoDeterminateSystems:mainfrom
detsys-pr-bot:detsys-ts-update-c7303495f43d348cac78091ef434443b1ef22485

Conversation

@detsys-pr-bot
Copy link
Copy Markdown

@detsys-pr-bot detsys-pr-bot commented Sep 3, 2025 •
edited by coderabbitai bot
Loading

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

Summary by CodeRabbit

  • Chores
    • Updated development tooling to latest versions to improve linting and type-checking consistency.
    • No user-facing changes; functionality and behavior remain unchanged.
    • No public API or interface changes.
    • Low-risk maintenance update.

@grahamc @github-actions
Update detsys-ts for: `Capture the version of Nix in addition to th…
bd38b77 
…e nix store version (DeterminateSystems#108)` (`c7303495f43d348cac78091ef434443b1ef22485`)
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Sep 3, 2025 •
edited
Loading

Walkthrough

Dev dependency versions updated in package.json: eslint-plugin-prettier to ^5.5.4 and typescript to ^5.9.2. No other files or public APIs changed.

Changes

Cohort / File(s) Summary
DevDependency bumps
package.json		 Update eslint-plugin-prettier ^5.5.3 → ^5.5.4; Update typescript ^5.8.3 → ^5.9.2

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

I hop through JSON fields so neat,
Two carrots bumped, a tidy treat—
Prettier shines, TypeScript anew,
Versions fresh as morning dew.
Thump-thump! My review is done—
Small hops today, then off I run. 🥕✨

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

  • MCP integration is disabled by default for public repositories
  • Jira integration is disabled by default for public repositories
  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between d6b4710 and bd38b77.

⛔ 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)
🔇 Additional comments (2)
package.json (2)

41-41: eslint-plugin-prettier bump looks good (peer deps satisfied).

v5.5.4 requires eslint ≥8 and prettier ≥3; you already have eslint ^8.57.1 and prettier ^3.6.2, so this upgrade is safe. (app.unpkg.com)

44-44: Keep TypeScript at 5.9.2 with @typescript-eslint v7.18.0
Running pnpm run lint produced no “unsupported TypeScript version” warnings or errors; you can safely keep "typescript": "^5.9.2" alongside "@typescript-eslint/eslint-plugin": "^7.18.0".

✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

  • Add @coderabbitai ignore or @coderabbit ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@grahamc grahamc enabled auto-merge September 3, 2025 15:33
@grahamc grahamc merged commit f81a117 into DeterminateSystems:main Sep 3, 2025
2 of 3 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Nov 7, 2025
Merged
@coderabbitai coderabbitai bot mentioned this pull request Jan 14, 2026
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