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 linter #200

Merged
merged 9 commits into from
Feb 13, 2022
Merged

Migrate linter #200

merged 9 commits into from
Feb 13, 2022

Conversation

thekevinscott
Copy link
Owner

TSLint is deprecated. This migrates over to use ESLint.

@codecov
Copy link

codecov bot commented Feb 13, 2022

Codecov Report

Merging #200 (367daf7) into ks/node (79374d3) will increase coverage by 5.47%.
The diff coverage is 92.64%.

Impacted file tree graph

@@             Coverage Diff             @@
##           ks/node     #200      +/-   ##
===========================================
+ Coverage    81.03%   86.51%   +5.47%     
===========================================
  Files            9        8       -1     
  Lines          290      304      +14     
  Branches        43       46       +3     
===========================================
+ Hits           235      263      +28     
+ Misses          55       41      -14     
Impacted Files Coverage Δ
packages/upscalerjs/src/models/idealo-gans.ts 33.33% <0.00%> (ø)
packages/upscalerjs/src/image.browser.ts 85.71% <85.71%> (ø)
packages/upscalerjs/src/loadModel.ts 89.09% <100.00%> (ø)
packages/upscalerjs/src/models.ts 100.00% <100.00%> (ø)
packages/upscalerjs/src/upscale.ts 91.04% <100.00%> (+0.74%) ⬆️
packages/upscalerjs/src/utils.ts 96.29% <100.00%> (+1.55%) ⬆️
packages/upscalerjs/src/warmup.ts 100.00% <100.00%> (ø)
packages/upscalerjs/src/tfjs.generated.ts

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 17fffd1...367daf7. Read the comment docs.

@thekevinscott thekevinscott merged commit a53c271 into ks/node Feb 13, 2022
@thekevinscott thekevinscott deleted the ks/migrate-linter branch February 13, 2022 19:56
thekevinscott added a commit that referenced this pull request Feb 27, 2022
* Expose Node package (#161)

* Set up Node build and Node integration tests

* Cut canary release (#179)

* Remove postinstall set:tfjs command (#182)

* Move set:tfjs to prepare step

* Publish next canary version

* Add Node Example to examples folder (#181)

* Update example script (#183)

* Support a GPU build (#184)

* Integration Tests for Loading Model in Browser (#186)

* Add tests for loading models via Node (#187)

* Add test to ensure model definitions are loaded in browser (#188)

* Add test to ensure model definitions are loaded in Node (#189)

* Bump versions

* Add image unit tests (#191)

* Separate image.ts into separate files and add it to the generation script (#192)

* Add integration test for upscaling a rank 4 tensor (#193)

* Leverage tf.browser.fromPixels on input (#194)

* Manipulated scaffolding code to ensure build processes are referencing the correct builds (#195)

* Move all but browser tests to leverage puppeteer (#196)

* Remove generated files from repo (#197)

* Do not include generated files in repo

* Add script to scaffold browser generated files after installing

* Scaffold on install (#198)

* Do not include generated files in repo

* Add script to scaffold browser generated files after installing

* Enforce minimum Node version (#199)

* Specify exact version of Node for unit tests, and add 14- and 16- specific Node integration tests (#201)

* Disable CI cache (#204)

* Update changelog (#203)

* Add umd field in package json (#202)

* Migrate linter (#200)

Migrate linter from tslint to eslint

* Bump lowest node support to 16 (#205)

* Bump lowest node support to 16

* Add support for image loading in Node (#206)

* Remove scaffolding step (#207)

* Add an integration test checking patch sizes (#211)

* Sets up in-browser unit testing for *.browser tests (#210)

* Add an example for loading a model locally in Node (#212)

* Fix leaking tensors (#213)

* Update package.json scripts to enable publishing (#214)

* Update documentation to include Node (#215)

* Update scripts for updating version across repo (#218)
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.

None yet

1 participant