Skip to content

Optional in/within prefix for the en deadline rule - #51

Open
dualfroz wants to merge 1 commit into
olebedev:masterfrom
dualfroz:fix/21-deadline-without-prefix
Open

dualfroz wants to merge 1 commit into
olebedev:masterfrom
dualfroz:fix/21-deadline-without-prefix

Conversation

@dualfroz

@dualfroz dualfroz commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

went with what you described, en.Deadline(rules.Override, en.DeadlineOptions{Aggressive: true}) makes the prefix optional so 2 hours matches too. default behaviour and en.All stay the same

Fixes #21

@dualfroz
dualfroz requested a review from olebedev as a code owner October 3, 2026 12:24
olebedev
olebedev previously approved these changes Oct 14, 2026
@dualfroz
dualfroz dismissed olebedev’s stale review October 14, 2026 07:46

The merge-base changed after approval.

@olebedev
olebedev self-requested a review October 14, 2026 07:47
olebedev
olebedev previously approved these changes Oct 14, 2026
@dualfroz
dualfroz dismissed olebedev’s stale review October 14, 2026 07:54

The merge-base changed after approval.

@olebedev
olebedev force-pushed the fix/21-deadline-without-prefix branch from da5fa90 to 8381199 Compare October 14, 2026 08:01
@olebedev

Copy link
Copy Markdown
Owner

Hi @dualfroz, there are merge conflicts in this branch, can you resolve them? Thanks.

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.

Is it possible to have 'deadlines' extended to work without 'in/within'

2 participants