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

Add hasOwn to section 3.7 #2610

Closed
wants to merge 4 commits into from
Closed

Add hasOwn to section 3.7 #2610

wants to merge 4 commits into from

Conversation

maurer2
Copy link

@maurer2 maurer2 commented Jun 29, 2022

Discussion related to this pr: #2606

@maurer2 maurer2 changed the title Add hasOwn to section 3.7 Add hasOwn to section 3.7 Jun 29, 2022
Copy link

@cyrus303 cyrus303 left a comment

Choose a reason for hiding this comment

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

upgrade looks good

Copy link
Collaborator

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

We should wait to land this until airbnb-js-shims is using https://npmjs.com/object.hasown or similar

@ljharb ljharb marked this pull request as draft August 18, 2022 15:31
@maurer2 maurer2 closed this by deleting the head repository Dec 14, 2022
@ljharb
Copy link
Collaborator

ljharb commented Dec 14, 2022

@maurer2 since you deleted the fork, this sadly can never be merged.

@maurer2
Copy link
Author

maurer2 commented Dec 14, 2022

Hello, totally forgot to close this before deleting the repo when doing some cleanup work. Sorry.

@ljharb
Copy link
Collaborator

ljharb commented Dec 15, 2022

Closing it first wouldn't have helped; until a PR is merged (or closed as a wontfix) the fork can't ever be deleted, or the PR is unrecoverable.

@maurer2 maurer2 reopened this Dec 15, 2022
@maurer2
Copy link
Author

maurer2 commented Dec 15, 2022

Okay, I see. Thanks I can recreate the pr and add it to the issue ticket, if it still makes sense to add this to the style guide. What do you think?

Cheers

@ljharb
Copy link
Collaborator

ljharb commented Dec 15, 2022

At this point let's just leave it as is; it's not clear if airbnb-js-shims is maintained or not.

@ljharb ljharb closed this Dec 15, 2022
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.

None yet

3 participants