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

Fixes TypeScript in upload-to-storage #2289

Merged
merged 1 commit into from
Feb 25, 2021
Merged

Conversation

gingi
Copy link
Member

@gingi gingi commented Feb 24, 2021

Used in the release pipeline.

Had not realized that the JS version of the file is being used in the release pipeline, and conversion to TS missed several imports and type definitions.

@codecov
Copy link

codecov bot commented Feb 25, 2021

Codecov Report

Merging #2289 (0b1ba8b) into master (5202b6f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2289   +/-   ##
=======================================
  Coverage   65.19%   65.19%           
=======================================
  Files         903      903           
  Lines       25712    25712           
  Branches     5105     5105           
=======================================
  Hits        16763    16763           
  Misses       8949     8949           

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 e39d67f...ff1212b. Read the comment docs.

Used in the release pipeline.
@gingi gingi force-pushed the bugfix/storage-upload-script branch from 57d5f66 to ff1212b Compare February 25, 2021 00:06
@gingi gingi merged commit 4ce1e1c into master Feb 25, 2021
@gingi gingi deleted the bugfix/storage-upload-script branch February 25, 2021 01:08
@cRui861 cRui861 linked an issue Apr 22, 2021 that may be closed by this pull request
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.

Switch to Typescript in upload-to-storage
2 participants