Skip to content

Finalizing 1.0 #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Finalizing 1.0 #6

wants to merge 4 commits into from

Conversation

Hexagon
Copy link
Member

@Hexagon Hexagon commented Jul 3, 2025

This pull request intends to finalize the initial draft of the full OCPS 1.0 specification. Background is available in the README.md.

  • Removes increments/OCPS-increment-1.0.md (as the first version is not an increment)
  • Adds specifications/OCPS-1.0.md

Aliases are intentionally postponed to next increment.

This PR is intended to open the document for community review and discussion before finalization. All feedback on clarity, overall concept, completeness, and any potential edge cases is welcome.

Please leave comments and suggestions directly on the PR.

@Hexagon Hexagon added this to the 1.0 milestone Jul 3, 2025
@Pinta365
Copy link
Member

Pinta365 commented Jul 9, 2025

While the current document states JAN-DEC and SUN-SAT are allowed, it relies on a shared, unstated assumption that everyone knows which name maps to which number. A core goal of a technical specification is to eliminate such assumptions.

So I would suggest we add a table under section 4.2 specifying this so there can be no confusion as to what text represent what number.

Thoughts?

Section 4.2:

  • Month and Day of Week names MUST be treated as case-insensitive.
  • In the Day of Week field, 0 and 7 MUST both be treated as Sunday.

Month Name Equivalents

Name Numeric Value
JAN 1
FEB 2
MAR 3
APR 4
MAY 5
JUN 6
JUL 7
AUG 8
SEP 9
OCT 10
NOV 11
DEC 12

Day of Week Name Equivalents

Name Numeric Value
SUN 0 or 7
MON 1
TUE 2
WED 3
THU 4
FRI 5
SAT 6

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