Skip to content

Comments

Use arrow functions for anonymous functions where possible (part #7)#19190

Merged
Josh-Cena merged 44 commits intomdn:mainfrom
teoli2003:function-arrow-7
Aug 8, 2022
Merged

Use arrow functions for anonymous functions where possible (part #7)#19190
Josh-Cena merged 44 commits intomdn:mainfrom
teoli2003:function-arrow-7

Conversation

@teoli2003
Copy link
Contributor

Arrow functions are shorter and easier to read (but don't bind this).

@github-actions github-actions bot added the Content:WebAPI Web API docs label Aug 5, 2022
@github-actions

This comment was marked as resolved.

@teoli2003 teoli2003 marked this pull request as ready for review August 5, 2022 13:33
@teoli2003 teoli2003 requested a review from a team as a code owner August 5, 2022 13:33
@teoli2003 teoli2003 requested review from wbamberg and removed request for a team August 5, 2022 13:33
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.

This one is ez!

@teoli2003
Copy link
Contributor Author

@Josh-Cena: I think I fixed all the comments (except one where I think it shouldn't).

@teoli2003 teoli2003 requested review from Josh-Cena and removed request for wbamberg August 8, 2022 10:22
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.

LGTM!

@Josh-Cena Josh-Cena merged commit 22c9372 into mdn:main Aug 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:WebAPI Web API docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants