Skip to content

Conversation

@smorimoto
Copy link
Member

@smorimoto smorimoto commented Dec 30, 2024

  • Fix the document filename
  • Link compatible tables directly
    • MDN for browsers and Node.js
    • Node.js official documentation for Node.js

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

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

Comments suppressed due to low confidence (4)

EMCASCRIPT.md:8

  • [nitpick] The term 'rest parameters' should be consistently capitalized as 'Rest Parameters' to match the other section titles.
### rest parameters

EMCASCRIPT.md:11

  • [nitpick] The term 'spread in function call' should be consistently capitalized as 'Spread Syntax (in function call)' to match the other section titles.
### spread in function call

EMCASCRIPT.md:17

  • [nitpick] The term 'ArrowFunction' should be consistently written as 'Arrow Function Expressions' to match the other section titles.
### ArrowFunction

EMCASCRIPT.md:27

  • The term 'polyfill-ed' should be corrected to 'Polyfilled' for clarity and consistency.
-- polyfill-ed in the repo

@hhugo
Copy link
Member

hhugo commented Dec 31, 2024

  • Fix the document filename

This is now on master. You can rebase and we'll hopefully get better diff for this PR

Signed-off-by: Sora Morimoto <sora@morimoto.io>
@smorimoto
Copy link
Member Author

Done

@hhugo hhugo merged commit 09e9842 into master Jan 1, 2025
29 checks passed
@hhugo hhugo deleted the update-es-doc branch January 1, 2025 08:08
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.

3 participants