Repository navigation
Moves the decimal lock line to 3.2.0 - #193
Merged
Merged
Conversation
mix.lock pinned decimal 3.1.1, which sits in the range of a published advisory on Decimal.round/3 (an unbounded allocation driven by the places argument). The lock now resolves 3.2.0, and mix hex.audit reports no retired or security advisory packages. Only the decimal line of mix.lock changes. The edge is a dev one: doctor requires decimal "~> 3.1" and jason lists it as optional, while mix.exs names no decimal requirement, so no requirement moves and no runtime dependency changes. No file under lib/ changes and no fragment is written; a published consumer resolves its own lock, so the change is invisible to anyone who only calls the public API. Refs: sui-nwkz
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Moves the one
decimalline ofmix.lockfrom 3.1.1 to 3.2.0.Why
3.1.1 is in the range of a published advisory on
Decimal.round/3(an unbounded allocation driven by theplacesargument).mix deps.getprinted it as vulnerable before the move and prints nothing about advisories after;mix hex.auditnow reports no retired or security advisory packages.What moves
mix.lock: thedecimalline only (git diff --stat: one line changed).mix deps.update decimalmoved nothing else.doctor(only: :dev) requiresdecimal "~> 3.1",jasonlists it as optional, andmix.exsnames nodecimalrequirement. 3.2.0 satisfies all of them.Changelog
No fragment.
changelog.d/README.mdis silent on lock-only moves; its test (could someone who only calls the public API tell the difference?) answers no, since no code, requirement or runtime dependency changes.Verification
mix qualitygreen on the branch: tests 1,214 of 1,214 passed, dialyzer, credo, doctor and docs clean.Provenance
Version choice (3.2.0, the one lock line) decided by the conductor under a standing consent, 2026-10-19.
Refs: sui-nwkz