Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

incremental clutz does not handle open interfaces #832

Open
rkirov opened this issue Jan 14, 2019 · 0 comments
Open

incremental clutz does not handle open interfaces #832

rkirov opened this issue Jan 14, 2019 · 0 comments

Comments

@rkirov
Copy link
Contributor

rkirov commented Jan 14, 2019

If a closure file wants to reopen an interface and add a property:

goog.module('goog.bar');

const I = goog.require('goog.foo.I');

/** @const {string} */
I.prototype.foo;

Incremental clutz currently completely ignores this, while potentially it can emit an appropriate TS interface extension.

declare namespace goog.foo.I {
  interface I {...}

Note, this would be tricky to implement because it would require an emit for 'goog.foo.I', while visiting 'goog.bar';

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant