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

fix(std/datetime): uppercase hour symbols #7107

Closed
wants to merge 280 commits into from
Closed

fix(std/datetime): uppercase hour symbols #7107

wants to merge 280 commits into from

Conversation

timreichen
Copy link
Contributor

Fix wrong symbol casing.
replace h and hh with H and HH respectively.
Lowercase is reserved for 12 hour days, uppercase for 24 hour days.

@caspervonb
Copy link
Contributor

caspervonb commented Aug 18, 2020

Lowercase is reserved for 12 hour days, uppercase for 24 hour days.

Based on what? in https://en.wikipedia.org/wiki/ISO_8601 hh is 24 hour format.

@timreichen
Copy link
Contributor Author

timreichen commented Aug 19, 2020

Based on what? in https://en.wikipedia.org/wiki/ISO_8601 hh is 24 hour format.

Yes, but based on http://www.unicode.org/reports/tr35/tr35-dates.html#Date_Field_Symbol_Table, the upper/lowercase convention is widely adopted in many different languages like swift and java. The formatter doesn't use ISO8601 symbols, because the ISO symbols are very limiting (for example: formatter has a for PM an AM parsing which doesn't exist in ISO8601).

This allows deno to extend the formatter if needed without having breaking changes.

@timreichen timreichen changed the title uppercase hour symbols fix(std/datetime): uppercase hour symbols Aug 22, 2020
kitsonk and others added 27 commits August 24, 2020 19:43
Co-authored-by: Bartek Iwańczuk <biwanczuk@gmail.com>
All benchmarks are done in Rust and can be invoked with
`cargo bench`.

Currently this has it's own "harness" that behaves like
`./tools/benchmark.py` did.
Because of this tests inside `cli/bench` are currently not run.
This should be switched to the language provided harness
once the `#[bench]` attribute has been stabilized.
caspervonb and others added 27 commits September 21, 2020 16:09
This removes the dependency on global state and instead relies on the
runtime's internal state to get the script sources it saw when it
collected code coverage for them.
This commit adds support for stack traces in "deno_core".

Implementation of "Display" trait for "JsError" has been updated
and in consequence "deno_core::js_check" became obsolete and
removed.
This encloses symbol keys when used in objects with brackets (e.g
[Symbol("Symbol.iterator")]).
This quotes and escapes symbol descriptions that contains characters
outside of the basic alpha-numeric identifier range.
@timreichen
Copy link
Contributor Author

close on favor of #7661

@timreichen timreichen closed this Sep 24, 2020
@timreichen timreichen deleted the fix-parser-symbol-names branch September 25, 2020 20:31
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.