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 expected output in example for Logical OR assignment (||=) - expressions-logical-or-assignment.js #2796

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

artemshar
Copy link
Contributor

Missing dot in expected output

Description

So as not to frustrate the reader, this operator is not a variation of the trim method.
When I first read about this method and this example confused me - “why did the dot disappear?”, but RUN put everything in its place.

Motivation

Perhaps these seconds of frustration will save thousands of developer hours. ;)

Additional details

Perhaps this will be my most significant contribution to open source in my entire life! (one dot in commit)

Related issues and pull requests

☺️

Missing dot in expected output

So as not to frustrate the reader, this operator is not a variation of the trim method
@artemshar artemshar requested a review from a team as a code owner June 22, 2024 12:31
@artemshar artemshar requested review from Josh-Cena and removed request for a team June 22, 2024 12:31
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.

@artemshar artemshar changed the title Update expressions-logical-or-assignment.js Fix expected output in example for Logical OR assignment (||=) - expressions-logical-or-assignment.js Jun 22, 2024
Copy link
Member

@Josh-Cena Josh-Cena left a comment

Choose a reason for hiding this comment

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

Thanks

@wbamberg wbamberg merged commit 64a3eb4 into mdn:main Jul 2, 2024
5 checks passed
Copy link

github-actions bot commented Jul 2, 2024

Congratulations on your first merged pull request. 🎉 Thank you for your contribution! Did you know we have a project board with high-impact contribution opportunities? We look forward to your next contribution.

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