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

Fixing the issues i mentioned in #311 & more consistent Coding-Style #312

Merged
merged 5 commits into from Jun 29, 2019

Conversation

hasezoey
Copy link
Contributor

Changes:

  • removing extra vscode task
  • adding "watch" to the npm scripts
  • adding tsbuildinfo / incremental to compile faster
  • adding "removeComments" to minimize output
  • TSLint Changes:
    • removing unnecessary options
    • adding/modifing options to get a more consistent coding style
    • apply changes from tslint

And the Changes to fix #311 mentioned Issues

adding "watch" to the npm scripts
adding "removeComments" to minimize output
removing unnecessary options
adding/modifing options to get a more consistent coding style
@hasezoey
Copy link
Contributor Author

btw @Ben305 please say me if DocumentPostFn<T> can be removed for tests to finish & remove because its not used

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 91.256% when pulling a41a86c on hasezoey:coderstyle into 6d5af8f on szokodiakos:master.

@Ben305
Copy link
Collaborator

Ben305 commented Jun 29, 2019

@hasezoey Thanks!

@Ben305 Ben305 merged commit dfa47eb into szokodiakos:master Jun 29, 2019
@hasezoey hasezoey deleted the coderstyle branch June 29, 2019 12:07
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

3 participants