Skip to content

[Transforms] Fix emit static property declaration in classExpression for down-level and ES6 target #8630

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

Closed
wants to merge 438 commits into from

Conversation

yuit
Copy link
Contributor

@yuit yuit commented May 16, 2016

Fix #8580 for both target >= ES6 and target < ES6

rbuckton and others added 30 commits April 4, 2016 13:01
select acceptable baselines for 'transformations' branch
…Indentation

Fixes indentation of class expression bodies
…ceInExternalModule

Fixes emit for exported namespaces in external modules.
…ypeForExponentiation

Fixed typo when computing transform flags for exponentation
…ines

Accepts nominal baseline changes for some failing tests.
update baselines in unittests, fix linter issues
* Update baselines

* Update project-baselines

* Revert "Update project-baselines"

This reverts commit eb16971.

* Remove incorrect baseline
Additional fixes for JSX transforms
sheetalkamat and others added 25 commits April 20, 2016 11:48
Correct export assignments for variables in transforms
[Transforms] Fix debug failure caused by merge.
[Transforms] Fix type reference directive tests by adding --module amd
* Do not emit leading comment of synthesized function expression in object literal property assignment

* Update baselines
* Use original text (e.g unicode) from source text when emit parameter-property declaration

* Update baseline
…tiesAndExportedModules

[Transforms] Fixes for ShorthandPropertyAssignments and exported namespaces
* Update baselines

* Add tests

* Update baselines from merging with transforms
Handel ImportEqualsDeclaration
…t declaration to require statement (#8321)

* Fix emit comment in wrong location when transform export declaration to require statement

* Add tests and baselines

* Add tests and baselines

* Update baselines
…8341)

* Remove the assertion as the assumption is not correct

* Add tests case and baselines
Update tests and baselines

Update tests and baselines

Update tests for downlevel emit

Add tests and update baselines

Add tests and baselines

Update baselines

Update baselines
…ts own static propertyDeclaration

Wip: attempt to fix down-level emit of class-expression with static property

Fix incorrect emit of referring to container classExpression inside its own static propertyDeclaration
@yuit yuit added Bug A bug in TypeScript Domain: Transforms Relates to the public transform API labels May 16, 2016
@yuit yuit closed this May 16, 2016
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug A bug in TypeScript Domain: Transforms Relates to the public transform API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants