Skip to content

Build & Release system#32

Merged
lolleko merged 4 commits intomasterfrom
build-system-update
Feb 16, 2018
Merged

Build & Release system#32
lolleko merged 4 commits intomasterfrom
build-system-update

Conversation

@lolleko
Copy link
Copy Markdown
Member

@lolleko lolleko commented Feb 16, 2018

Things that have to be done before this can be merged:

  • Add npm api key (encrypted in travis.yml or ENV_VAR via travis web interface)
  • Remove js files from dist/ they are already ignored but need to be removed aswell
  • Add License to repo and package.json

After merge:

  • Push first release :D

Release system:

npm run release-minor (alternatives: release-patch or release-major)
will trigger npm version which will build & test locally, if those steps succeed a tag is created and pushed to GitHub.
Travis will then build & test and npm publish the repo.

Things that should be done in the near Future:

  • Improve Readme and documentation (especially docs for custom decorators)

  • Change tstl (Compiler.js) command line interface to be similar (or equal) to tsc

  • Add coveralls support for coverage tracking

  • Improve lualib integration: Right now lualib is a bit hidden in the dist/ folder and will be even lesss visible in npm packages. Therefore we should automatically require lualib into projects. For that we will need to include the typings (lib-typescript.d.ts) when typescript is parsing the source file (before compilation).

Then we need to add the lualib functions to the output (during compilation), 2 possible approaches for that are:

  1. Keep track of functions/classes from lualib that are used and just copy them into the Lua source file where they are used e.g: local TS_Splice = function() .....

  2. Copy typescript.lua to output root directory of the compiled project. and require('typescript') in each file where the lib is used. (It would be better if we changed typescript.lua to a module structure that way we can do local = require, and utilise the module cache)

Closes #17

@lolleko lolleko merged commit e9f254c into master Feb 16, 2018
@Perryvw Perryvw deleted the build-system-update branch February 18, 2018 11:33
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.

1 participant