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
Handle interfaces in the generate-externs task #2643
Conversation
} | ||
}); | ||
|
||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"No newline at end of file"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
d34f86c
to
e560f52
Compare
@@ -28,6 +28,7 @@ exports.publish = function(data, opts) { | |||
[ | |||
{define: {isObject: true}}, | |||
{api: {isString: true}}, | |||
{'interface': {is: true}}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/me discovered that interface
is a reserved keyword.
16b98df
to
e560f52
Compare
Is this good to merge? When merged, it will be easier to fix the remaining issues in #2644. |
I'm fine with this, as long as it does not break the doc generation. |
This does not affect the doc generation at all. |
Handle interfaces in the generate-externs task
See #2615.