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

Support more of the export syntax #125

Closed
wants to merge 2 commits into from

Conversation

resolritter
Copy link
Contributor

Covers

  • (abstract)? class
  • interface

as per microsoft/TypeScript#3792 (comment) (2020/12/15)

other constructs are not yet supported by TypeScript itself, as pointed out in microsoft/TypeScript#18628 (comment)


closes #100

@mjambon
Copy link
Contributor

mjambon commented Jan 29, 2021

Looks great. Thank you! @maxbrunsfeld, can we merge this?

@resolritter resolritter force-pushed the exports branch 2 times, most recently from 27319bd to ad50357 Compare January 29, 2021 05:11
@aryx
Copy link
Contributor

aryx commented Feb 23, 2021

There are lots of conflicts. @resolritter you probably need to merge/rebase master.

@resolritter
Copy link
Contributor Author

@maxbrunsfeld it should be ready

@aryx
Copy link
Contributor

aryx commented Mar 3, 2021

thx @resolritter !

@mjambon
Copy link
Contributor

mjambon commented Mar 9, 2021

We'd like merge this, @maxbrunsfeld. It would avoid us from having to use a fork that includes pending PRs, which gets quickly confusing. thx!

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.

Support 'export default abstract class C {}'
5 participants