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

Semantic exmple #2464

Closed
wants to merge 2 commits into from
Closed

Semantic exmple #2464

wants to merge 2 commits into from

Conversation

jdavidleon
Copy link

Description

Semantic exmple

@github-actions
Copy link

It looks like this is your first pull request. 🎉 Thank you for your contribution! One of the project maintainers will triage and assign the pull request for review. We appreciate your patience. To safeguard the health of the project, please take a moment to read our code of conduct.

@NiedziolkaMichal
Copy link
Member

Thank you for trying to contribute, but this is not a good improvement:

  • Your example throws Error: people1 is not defined
  • (4) shouldn't be added after Array, because it will not match the actual output
  • Empty line after work property is not needed
  • Property names a, b, c are perfectly clear, there is no need to change them

@jdavidleon
Copy link
Author

Hi. Can you tell me how to find examples that are not in the MDN documentation. In order for me to collaborate, I was reading the ECMA 2022 documentation but they are already done.

Or how can I collaborate on something. I learned to program but I have never worked in a team and I would like to do it.

@NiedziolkaMichal
Copy link
Member

I am glad to hear that. You can find missing examples in #2374 and #2463

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