Skip to content
This repository has been archived by the owner on Nov 24, 2021. It is now read-only.

Refactor and use TypeScript: #30

Merged
merged 5 commits into from
Oct 18, 2019
Merged

Refactor and use TypeScript: #30

merged 5 commits into from
Oct 18, 2019

Conversation

intelliot
Copy link
Contributor

  • Migrate to TypeScript and TSLint
  • Vendor ripple-address-codec (move dependency into this project)
  • Add support for Travis CI (.travis.yml)
  • Use "dist/*" for distribution files
  • Bump version to 1.0.0-beta.0
  • Add yarn.lock

- Migrate to TypeScript and TSLint
- Vendor ripple-address-codec (move dependency into this project)
- Add support for Travis CI (.travis.yml)
- Use "dist/*" for distribution files
- Bump version to 1.0.0-beta.0
- Add yarn.lock
@sublimator
Copy link
Contributor

sublimator commented Jan 23, 2019 via email

@intelliot
Copy link
Contributor Author

To simplify, reducing the number of places to look for things

},
"scripts": {
"compile": "babel --optional runtime -d distrib/npm/ src/",
"compile": "tsc",
"compile-babel": "babel --optional runtime -d distrib/npm/ src/",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why two compiles? Can we get rid of Babel now that we're using TS?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point - yes, I think we can get rid of Babel

@FredKSchott
Copy link

LGTM!

@sublimator
Copy link
Contributor

sublimator commented Jan 24, 2019 via email

@intelliot
Copy link
Contributor Author

I think we'll expose the necessary functionality from ripple-keypairs

@sublimator
Copy link
Contributor

sublimator commented Jan 25, 2019 via email

package.json Outdated
"distrib/npm/*",
"bin/*",
"build/*",
"dist/*",
"test/*"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can this also be removed/is it necessary to publish the tests to NPM?

@intelliot intelliot mentioned this pull request Oct 17, 2019
@intelliot intelliot merged commit a23cda3 into master Oct 18, 2019
@intelliot intelliot deleted the refactor branch February 12, 2020 05:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants