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

Update ECMASCRIPT.md #1851

Merged
merged 2 commits into from
Mar 6, 2025
Merged

Update ECMASCRIPT.md #1851

merged 2 commits into from
Mar 6, 2025

Conversation

smorimoto
Copy link
Member

This may be a bit too much, but the current runtime dependencies are now truly exhaustive.

Signed-off-by: Sora Morimoto <sora@morimoto.io>

Choose a reason for hiding this comment

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

PR Overview

This PR updates ECMASCRIPT.md to document a more exhaustive list of runtime dependencies and ECMAScript features.

  • Added several new feature sections (e.g., Arrow function expressions, const, for...of, let, WeakMap, BigInt, Optional chaining)
  • Introduced new Web API compatibility sections (e.g., Canvas API, Crypto API, Document API)

Reviewed Changes

File Description
ECMASCRIPT.md Reorganized ECMAScript feature headings, added new features documentation, and introduced several Web API compatibility links.

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (3)

ECMASCRIPT.md:17

  • The section heading was changed from 'Rest parameters' to 'const', but a 'Rest parameters' section appears later. Please verify if this reordering is intentional or if one of the headings needs to be corrected.
### const

ECMASCRIPT.md:11

  • [nitpick] The note 'For Wasm_of_ocaml' appears repeatedly. Consider standardizing the formatting or clarifying its purpose to avoid redundancy.
- For Wasm_of_ocaml

ECMASCRIPT.md:55

  • The change from a 'Map' section (which included a runtime usage note) to 'WeakMap' could be confusing. Please check if this replacement was intended or if both sections should coexist with clear descriptions.
### WeakMap
@hhugo hhugo merged commit 1aac41e into master Mar 6, 2025
30 of 31 checks passed
@hhugo hhugo deleted the update-es-doc branch March 6, 2025 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants