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

Get latest changes from dart-archive/ts2dart master #1

Open
wants to merge 110 commits into
base: master
Choose a base branch
from

Conversation

devinrhode2
Copy link

I'm naively thinking that somehow ts2dart could be brought back to life. I'm comfortable with TS and would love to be able to fiddle around with converting TS code to dart. I noticed your fork due to the Lovely Forks Chrome extension. jumperchen/ts2dart has a few more commits than origin/master, and merging this would also help others see that there's still some interest in this project.

vikerman and others added 30 commits February 1, 2016 14:56
mprobst and others added 28 commits April 25, 2016 13:18
Summary:
Otherwise this task could run without its prerequisites.

feat: handle lib.d.ts definitions from tsconfig

When configured through tsconfig, instead of a custom compiler host,
the fileName for default library definitions is the absolute path of the
library.

This change explicitly uses the compiler host's information for the
location of lib.d.ts.

Reviewers: alexeagle, mhevery

Reviewed By: mhevery

Subscribers: typescript-eng

Differential Revision: https://reviews.angular.io/D80
Summary:
This allows marking an expression as const without JavaScript code for
the `CONST_EXPR` call. `CONST_EXPR` is not recognised by simple bundlers
such as rollup as a side-effect free function, which causes unnecessary
symbols to be retained in the binary. Avoiding `CONST_EXPR` and instead
using `/* @ts2dart_const */ (...)` should thus allow Angular 2 to reduce
its binary size.

Reviewers: mhevery, alexeagle

Reviewed By: alexeagle

Subscribers: typescript-eng

Differential Revision: https://reviews.angular.io/D83
Summary: ... to allow better results in rollup.

Reviewers: mhevery, alexeagle

Reviewed By: alexeagle

Subscribers: typescript-eng

Differential Revision: https://reviews.angular.io/D85
Reviewers: alexeagle, mhevery

Reviewed By: mhevery

Subscribers: typescript-eng

Differential Revision: https://reviews.angular.io/D86
Summary:
Installing `test` version 0.12.8 fails on Travis, but works locally. Version
0.12.13 works in both locations.

This does not really make sense - `pub get` should be deterministic, and should
also be able to get the older version of the test package, but whatever.

Reviewers: vikerman

Reviewed By: vikerman

Subscribers: typescript-eng

Differential Revision: https://reviews.angular.io/D87
Summary:
This is useful when the feature branch tracks a different upstream branch from
master, which happens e.g. when you `git push --set-upstream feature feature`,
which in turn is useful to have Travis test a change.

Reviewers: alexeagle

Subscribers: typescript-eng

Differential Revision: https://reviews.angular.io/D88
Summary: In Dart, all default values have to be const.

Reviewers: mhevery

Reviewed By: mhevery

Subscribers: typescript-eng

Differential Revision: https://reviews.angular.io/D89
Summary: This fixes the bug where all code in const marked classes was treated as const.

Reviewers: mhevery

Reviewed By: mhevery

Subscribers: typescript-eng

Differential Revision: https://reviews.angular.io/D90
fix: Future.{value,error} are named constructors, not statics
When using TypeScript's path support and driving ts2dart using a tsconfig.json,
path resolution yields absolute paths.

This change resolves the well known paths using TypeScript's own module
resolver, and then looks up absolute file paths against that, greatly
simplifying the path handling logic (woohoo!).

Reviewers: alexeagle

Subscribers: typescript-eng

Differential Revision: https://reviews.angular.io/D94
The last entry in the array is the code under test for multi-file
test cases.
These are used to mark object literals as providers that need to
be translated to `new Provider(...)` in Dart.
TypeScript resolves files to absolute path names only when they
are discovered through an import. To avoid that mess, resolve all
paths before proceeding.

Also port the remaining e2e test to use a tsconfig.json.
Summary:
Previously only worked for functions.

Fixes dart-archive#282.

Reviewers: tbosch

Reviewed By: tbosch

Subscribers: typescript-eng

Differential Revision: https://reviews.angular.io/D100
…-archive#372)

* Add ability to install globally and execute as command line app

* reformat code

* add showHelp method and upgrade test to 0.12.15+3 in pubspec.yaml

* correct typo
@devinrhode2
Copy link
Author

devinrhode2 commented Dec 24, 2020

If anyone wants to fiddle around with ts2dart, obviously https://github.com/dart-archive/ts2dart is not a bad version to play around with. @jumperchen's fork keikai has a few extra changes that may be of interest: https://github.com/jumperchen/ts2dart/tree/keikai

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.