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 typo and broken links #2347

Merged
merged 2 commits into from
Jul 7, 2022
Merged

Conversation

tars0x9752
Copy link
Contributor

Fix broken links in docs/ja and a minor typo fix in docs/zoneddatetime.md.

@@ -224,7 +224,7 @@ date.daysInYear; // => 365

## 文字列による永続性

すべての`Temporal`タイプは、永続性や相互運用性のために文字列による表現を持っています。各タイプと文字列表現の対応を以下に示します。Temporal で使用されている ISO 8601 や RFC 3339 に関する、より詳細な情報と標準化に向けた取り組みに関しては[ECMAScript 拡張の ISO-8601 と RFC 3339](./strings.md)を参照してください。
すべての`Temporal`タイプは、永続性や相互運用性のために文字列による表現を持っています。各タイプと文字列表現の対応を以下に示します。Temporal で使用されている ISO 8601 や RFC 3339 に関する、より詳細な情報と標準化に向けた取り組みに関しては[ECMAScript 拡張の ISO-8601 と RFC 3339](./iso-string-ext.md)を参照してください。
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Perhaps this could be the ../string.md which is the original English one. But I guess the translator intended to use ./iso-string-ext.md here. In the context of the translation, ./iso-string-ext.md seems to be fine to me. (I'm new to here so I'm not completely sure which one it should be though.)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Either one is fine, I trust your judgement about what makes the most sense in this translation.

Copy link
Collaborator

Choose a reason for hiding this comment

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

iso-string-ext.md looks like it was the original filename of strings.md before I renamed it in #2059. So if we want to show the Japanese page (even if the content is somewhat out of date) then ./iso-string-ext.md is correct. If we want to link to the latest content, even though it's English, then I assume that we should use ../strings.md ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, I just opened another PR #2353 to add a note to docs/ja about the latest content and leave the Japanese ./iso-string-ext.md page as it is.

Copy link
Collaborator

@ptomato ptomato left a comment

Choose a reason for hiding this comment

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

Thanks!

@ptomato ptomato enabled auto-merge (rebase) July 7, 2022 18:32
@codecov
Copy link

codecov bot commented Jul 7, 2022

Codecov Report

Merging #2347 (049985a) into main (5b38ab4) will decrease coverage by 9.32%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2347      +/-   ##
==========================================
- Coverage   91.09%   81.76%   -9.33%     
==========================================
  Files          19       17       -2     
  Lines       10573    10554      -19     
  Branches     1697     1480     -217     
==========================================
- Hits         9631     8630    -1001     
- Misses        932     1880     +948     
- Partials       10       44      +34     
Flag Coverage Δ
test262 ?
tests 81.76% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
polyfill/lib/plainyearmonth.mjs 43.36% <0.00%> (-33.68%) ⬇️
polyfill/lib/plaintime.mjs 37.82% <0.00%> (-31.38%) ⬇️
polyfill/lib/plainmonthday.mjs 51.06% <0.00%> (-31.21%) ⬇️
polyfill/lib/plaindate.mjs 56.06% <0.00%> (-26.89%) ⬇️
polyfill/lib/plaindatetime.mjs 53.26% <0.00%> (-25.48%) ⬇️
polyfill/lib/instant.mjs 75.12% <0.00%> (-15.43%) ⬇️
polyfill/lib/zoneddatetime.mjs 87.14% <0.00%> (-11.10%) ⬇️
polyfill/lib/duration.mjs 69.93% <0.00%> (-10.68%) ⬇️
polyfill/lib/now.mjs 91.22% <0.00%> (-8.78%) ⬇️
polyfill/lib/timezone.mjs 85.35% <0.00%> (-8.29%) ⬇️
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5b38ab4...049985a. Read the comment docs.

@ptomato ptomato merged commit d74e70e into tc39:main Jul 7, 2022
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.

3 participants