Skip to content

DO NOT MERGE: months = { from, to } closed table - #396

Draft
yyolk wants to merge 1 commit into
masterfrom
cursor/months-from-to-closed-table-8ea5
Draft

yyolk wants to merge 1 commit into
masterfrom
cursor/months-from-to-closed-table-8ea5

Conversation

@yyolk

@yyolk yyolk commented Sep 15, 2026

Copy link
Copy Markdown
Owner

Look-later draft — do not merge. Parks an inclusive closed-table form for pressed months next to the existing int list.

[months]
from = 1
to = 12

or inline months = { from = 1, to = 3 }.

_parse_months expands the table to tuple[int, ...] at parse time. No new Spec type.

  • omit / missing → full year 1..12
  • months = [1, 2, 3] stays a literal member list (a 2-element list is not a span)
  • month = N still maps to (N,)
  • table: both from and to required ints, 1 <= from <= to <= 12, unknown keys fail (unknown months key …)
  • no "1..12" strings, no step, no multi-span

examples/nomad.toml keeps the explicit list and notes the table equivalent in a comment.

Open in Web Open in Cursor 

Accept an inclusive [months] from/to table alongside the existing
int list, omit (full year), and legacy month = N.

Co-authored-by: Joseph Chiocchi <joe@yolk.cc>
@github-actions github-actions Bot added the DO NOT MERGE Do not merge; workflow forces this PR to draft on the default branch label Sep 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Forced this pull request to draft because the title contains "DO NOT MERGE" has a "DO NOT MERGE" label.

Drafts cannot be merged. Remove DO NOT MERGE from the title, then click Ready for review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DO NOT MERGE Do not merge; workflow forces this PR to draft on the default branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant