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 closure annotation for all paths #4786

Merged
merged 1 commit into from
Aug 11, 2017
Merged

Update closure annotation for all paths #4786

merged 1 commit into from
Aug 11, 2017

Conversation

beckysiegel
Copy link
Contributor

Polymer.Path.get accepts both a string path or an Array path, so
functions that call this should allow for either as well. Already
changed for Polymer.prototype.push here:

#4776

Reference Issue

@dfreedm
Copy link
Member

dfreedm commented Aug 10, 2017

@beckysiegel Oh sorry, closure-types.js is a generated file from gulp generate-closure-externs!
I've already updated that file with the changes from your PR (#4776) in c20b657

@dfreedm dfreedm closed this Aug 10, 2017
@dfreedm
Copy link
Member

dfreedm commented Aug 10, 2017

Oops, my mistake, this affects more than that file 😨

@dfreedm dfreedm reopened this Aug 10, 2017
Copy link
Member

@dfreedm dfreedm left a comment

Choose a reason for hiding this comment

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

The changes in property-effects look good to me, but you should run gulp generate-closure-externs to update the closure-types file.

Thanks!

functions that call this should allow for either as well. Already
changed for Polymer.prototype.push here:

#4776
@beckysiegel
Copy link
Contributor Author

@azakus done!

@dfreedm dfreedm merged commit 42fd830 into Polymer:master Aug 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants