Skip to content

chore: upgrade decimal to 3.1 - #91

Merged
cpursley merged 1 commit into
agoodway:masterfrom
adriencohen:chore/decimal-3
May 28, 2026
Merged

cpursley merged 1 commit into
agoodway:masterfrom
adriencohen:chore/decimal-3

Conversation

@adriencohen

@adriencohen adriencohen commented May 28, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores
    • Updated decimal library dependency to version 3.1

Review Change Stack

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@coderabbitai

coderabbitai Bot commented May 28, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: eb2c6f9d-f12d-4409-905a-c776ece4e307

📥 Commits

Reviewing files that changed from the base of the PR and between d841a8e and cceacb0.

⛔ Files ignored due to path filters (1)
  • mix.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • mix.exs

📝 Walkthrough

Walkthrough

The pull request updates the :decimal dependency constraint in mix.exs from version ~> 2.3.0 to ~> 3.1, allowing the project to use Decimal 3.1 instead of the earlier 2.3 range.

Changes

Decimal Dependency Update

Layer / File(s) Summary
Decimal dependency version constraint
mix.exs
The :decimal dependency version is updated from ~> 2.3.0 to ~> 3.1 in the deps/0 function, allowing compatibility with Decimal 3.1 releases.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A hop and a bump, the decimal's new,
From two-point-three to three-point-one true,
Dependencies dance in the mix.exs file,
A simple update to keep pace with style!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: upgrading the decimal dependency from version 2.3.0 to 3.1.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

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

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

@cpursley
cpursley merged commit a1e2b03 into agoodway:master May 28, 2026
1 of 2 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