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

Migrate to Typescript #119

Closed
philipmjohnson opened this issue May 11, 2017 · 1 comment
Closed

Migrate to Typescript #119

philipmjohnson opened this issue May 11, 2017 · 1 comment
Assignees

Comments

@philipmjohnson
Copy link
Contributor

philipmjohnson commented May 11, 2017

Typescript appears to be the de facto standard for typed JS, and moving to it in RadGrad should:

  1. Reduce the number of currently hidden type-related errors present in our codebase.
  2. Enable us to catch type-related errors at compile time, speeding up development.
  3. Provide type-level documentation about the code, making maintenance and developer on-boarding easier.

The costs are:

  1. We have to spend time to learn Typescript.
  2. We have to spend time to convert our current codebase.

Given that one thing I wanted to do this summer is a walkthrough of the codebase, it occurs to me that we could combine the two activities by having each of use read some one else's code, and at the same time convert it to Typescript.

Here are some helpful resources to get started:

If this seems like a good idea, I would create a branch to do an initial setup of Typescript and convert some code to complete this issue. Then we can create further issues to migrate the remainder of the codebase.

@philipmjohnson philipmjohnson changed the title Move to typescript Migrate to Typescript May 11, 2017
@philipmjohnson philipmjohnson self-assigned this May 11, 2017
@philipmjohnson philipmjohnson added this to the M3 milestone May 11, 2017
@philipmjohnson philipmjohnson modified the milestones: M5, M3 Aug 9, 2017
@philipmjohnson philipmjohnson added this to Backlog in M5 Aug 9, 2017
@philipmjohnson philipmjohnson removed this from Backlog in M5 Sep 12, 2017
@philipmjohnson philipmjohnson modified the milestones: M7, M5 Sep 12, 2017
@cammoore cammoore added this to Backlog in Fall 2018 Sep 14, 2017
@philipmjohnson philipmjohnson removed this from the M7 milestone Jan 29, 2018
@philipmjohnson philipmjohnson removed this from Backlog in Fall 2018 Jun 18, 2018
@cammoore
Copy link
Contributor

See RadGrad2.

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

No branches or pull requests

2 participants