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

build: Use typescript, update webpack, linters etc. #78

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vire
Copy link
Collaborator

@vire vire commented Apr 3, 2017

  • fix .editorconfig - CRLF -> LF
  • lint: semi set to never
  • lint: comma dange - always-multiline, because of duplicating lines
  • moved jest config into .jestrc
  • update package.json scripts
  • added tsconfig, tslint, tsfmt for styling codebase
  • migrated impl to ts
  • migrated most tests to ts

@vire vire force-pushed the migrate-to-typescript branch 2 times, most recently from cbfd9fb to 9f0c6a7 Compare April 3, 2017 11:57
@codecov-io
Copy link

codecov-io commented Apr 3, 2017

Codecov Report

Merging #78 into master will decrease coverage by 3.11%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #78      +/-   ##
==========================================
- Coverage   15.05%   11.94%   -3.12%     
==========================================
  Files          14        7       -7     
  Lines         352      134     -218     
  Branches       45        9      -36     
==========================================
- Hits           53       16      -37     
+ Misses        299      118     -181
Impacted Files Coverage Δ
src/js/sites/fixtures/maxireality.js 100% <ø> (ø) ⬆️
src/js/sites/fixtures/bezrealitky.js 100% <ø> (ø) ⬆️
src/js/sites/fixtures/sreality.js 100% <ø> (ø) ⬆️
src/js/sites/fixtures/realityidnes.js 100% <ø> (ø) ⬆️
src/js/components/Availability.vue 3.38% <0%> (+3.38%) ⬆️
src/js/components/App.vue 10.6% <0%> (+10.6%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 37a5b8a...e86bc12. Read the comment docs.

@vire vire force-pushed the migrate-to-typescript branch from 9f0c6a7 to c3de6ca Compare April 3, 2017 12:46
* added 🐶husky to run prepush
* fix .editorconfig - corrected to use LF setting for all files
* lint: semi set to never
* lint: comma dange - always-multiline, because of duplicating lines
* moved jest config into .jestrc
* update package.json scripts
* added tsconfig, tslint, tsfmt for styling codebase
* migrated all impl to ts
* migrated non-vue tests to ts

Closes realreality#74 realreality#75
@vire vire force-pushed the migrate-to-typescript branch from c3de6ca to e86bc12 Compare April 3, 2017 22:36
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.

2 participants