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

Update core.js types #5067

Closed
wants to merge 1 commit into from
Closed

Conversation

apsavin
Copy link
Contributor

@apsavin apsavin commented Oct 10, 2017

Add PropertyDescriptor, update Object, Function, Proxy and setTimeout/setInterval

@apsavin
Copy link
Contributor Author

apsavin commented Oct 11, 2017

@calebmer Sorry for disturbing, but maybe you can help me with this PR.
I think that AppVeyor is wrong and this PR is ok to merge. What can I do to fix the build and merge the PR?

@gabelevi
Copy link
Contributor

Yeah, AppVeyor has been a little flakey recently. I'll try and fix it soon. In the meantime, I'll get this PR tested and reviewed!

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@gabelevi has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@lll000111
Copy link
Contributor

lll000111 commented Feb 14, 2018

Just how annoying are "pings" (reminder comments with no new information — like this one)? On the other hand, I cannot use getters/setters at all right now (without $FlowFixMe comment)?

Related PR: #4788

@apsavin apsavin deleted the core-js-types branch May 3, 2018 16:06
@bakkot bakkot mentioned this pull request May 16, 2018
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.

None yet

5 participants