Skip to content
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

editorial: Remove the definition of "epoch-relative timestamp". #153

Merged
merged 3 commits into from
Mar 21, 2023

Conversation

jyasskin
Copy link
Member

@jyasskin jyasskin commented Mar 20, 2023

It was ambiguous about whether it rounded to the nearest integer, and new specifications should use the terms in #sec-tools instead.

whatwg/notifications#192 removes the only use listed in https://dontcallmedom.github.io/webdex/e.html#epoch-relative%20timestamp%40%40hr-time%25%25dfn. If I missed another one, whoever cleans that up can imitate the replacement in that PR or in the #the-epochtimestamp-typedef section here.


Person merging, please make sure that commits are squashed with one of the following as a commit message prefix:

  • chore: fixes unrelated to the spec itself (e.g., fix to Github action, html tidy, spec config, etc.).
  • editorial: a non-normative change to the spec (e.g., fixing an example or typo, adding a note).
  • change: a normative change to existing engine behavior.
  • And use none if it's a new normative requirement.

If this is a "change", please explain what's significantly changing.
In particular, if the change results in potential backwards compatibility issues and content breakage, it needs to be justified.

For normative spec changes, please get implementation commitments anf file issues on the various engines during the review process. All tasks should be complete before merging.

Implementation commitment - primarily for "change" and other normative changes:

Tasks:


Preview | Diff

It was ambiguous about whether it rounded to the nearest integer, and
new specifications should use the terms in #sec-tools instead.

If I missed cleaning up some existing use, imitate the replacement in
the #the-epochtimestamp-typedef section.
index.html Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
Copy link
Contributor

@noamr noamr left a comment

Choose a reason for hiding this comment

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

LGTM % nit

@noamr noamr merged commit 85a2ba1 into w3c:gh-pages Mar 21, 2023
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