Skip to content

AG-306 - Rename MultipleAcquisitionAction.OFF to LENIENT for clarity#185

Merged
barreiro merged 1 commit intoagroal:masterfrom
gastaldi:rename_multiple_aquisition
May 4, 2026
Merged

AG-306 - Rename MultipleAcquisitionAction.OFF to LENIENT for clarity#185
barreiro merged 1 commit intoagroal:masterfrom
gastaldi:rename_multiple_aquisition

Conversation

@gastaldi
Copy link
Copy Markdown
Collaborator

@gastaldi gastaldi commented Apr 28, 2026

LENIENT better conveys the intent (multiple acquisitions allowed) and forms a natural severity spectrum with WARN and STRICT.

The deprecated OFF constant is preserved for backward compatibility and normalized to LENIENT in the setter.

@gastaldi gastaldi requested a review from barreiro April 28, 2026 15:55
@gastaldi gastaldi changed the title Rename MultipleAcquisitionAction.OFF to LENIENT for clarity Rename MultipleAcquisitionAction.OFF to LENIENT for clarity Apr 28, 2026
LENIENT better conveys the intent (multiple acquisitions allowed) and
forms a natural severity spectrum with WARN and STRICT. The deprecated
OFF constant is preserved for backward compatibility and normalized to
LENIENT in the setter.
@gastaldi gastaldi force-pushed the rename_multiple_aquisition branch from 750de0f to ad63435 Compare April 29, 2026 23:19
Copy link
Copy Markdown
Contributor

@barreiro barreiro left a comment

Choose a reason for hiding this comment

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

please create an issue for it !!

@gastaldi gastaldi changed the title Rename MultipleAcquisitionAction.OFF to LENIENT for clarity AG-306 - Rename MultipleAcquisitionAction.OFF to LENIENT for clarity May 1, 2026
@barreiro barreiro merged commit a839863 into agroal:master May 4, 2026
8 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