Skip to content

Maintenance#30

Merged
timohanke merged 3 commits into
mainfrom
maintenance
Jan 24, 2026
Merged

Maintenance#30
timohanke merged 3 commits into
mainfrom
maintenance

Conversation

@timohanke
Copy link
Copy Markdown
Contributor

@timohanke timohanke commented Jan 24, 2026

  • Bump dev-dependencies to ones using core
  • Prepare for use with mops CLI 2.0.1

Summary by CodeRabbit

  • Chores
    • Bumped package version to 0.4.4.
    • Updated development dependencies (bench → 2.0.0; added core 2.0.0; removed test).
    • Updated toolchain (moc → 1.1.0).
  • Documentation
    • Added changelog entry for v0.4.4 noting dependency updates and removal of unused dev dependency.

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

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 24, 2026

📝 Walkthrough

Walkthrough

This PR updates package metadata: bumps project version to 0.4.4, adjusts dev dependencies (removes test, upgrades bench, adds core) and updates the moc toolchain. Changelog entry added.

Changes

Cohort / File(s) Summary
Dependency & Toolchain Updates
mops.toml
Bump package version 0.4.3 → 0.4.4; remove dev dependency test; upgrade dev dependency bench 1.0.0 → 2.0.0; add dev dependency core = "2.0.0"; update moc toolchain 1.0.0 → 1.1.0.
Changelog
CHANGELOG.md
Add entry for v0.4.4 noting bench bump and removal of test dev dependency.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 I hopped through toml and changelog bright,

versions nudged gently into the light.
Bench grew stronger, test took flight,
Moc stretched a paw, core joined the night.
A tiny update — all snug and right.

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Maintenance' is vague and generic, failing to convey the specific changes made in the PR (dependency updates and mops CLI 2.0.1 preparation). Consider using a more descriptive title like 'Bump dev dependencies for mops CLI 2.0.1 compatibility' to clearly communicate the main purpose of the changes.
✅ 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.


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

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@mops.toml`:
- Around line 12-15: The mops.toml currently pins dev-dependency core = "2.0.0"
which may not exist on the mops registry; verify whether version 2.0.0 of the
core package is published and available, and if not either change the dependency
to the known-published version (e.g., core = "1.0.0") or publish/release core
v2.0.0 to the registry; after updating mops.toml (the core = "2.0.0" entry) run
a local mops build/resolve to confirm the dependency resolves and CI will
succeed.

Comment thread mops.toml
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the development dependencies and toolchain versions to prepare the package for use with mops CLI 2.0.1.

Changes:

  • Removed the unused "test" package (was version 2.1.1) from dev-dependencies
  • Added "core" package version 2.0.0 to dev-dependencies
  • Updated "bench" package from version 1.0.0 to 2.0.0
  • Updated "moc" toolchain from version 1.0.0 to 1.1.0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@CHANGELOG.md`:
- Around line 3-6: Update the "## 0.4.4" section in CHANGELOG.md to include the
missing items from the PR: add a bullet mentioning the addition/bump of the core
dev dependency (refer to the 0.4.4 header) and a bullet noting the moc toolchain
update; keep phrasing consistent with existing bullets (e.g., "* Add/Update core
dev dependency" and "* Bump moc toolchain to <new-version>") so the changelog
fully reflects the PR changes.

Comment thread CHANGELOG.md
@timohanke timohanke merged commit 373d84d into main Jan 24, 2026
2 checks passed
@timohanke timohanke deleted the maintenance branch January 24, 2026 21:50
@coderabbitai coderabbitai Bot mentioned this pull request Mar 18, 2026
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