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

feat: Live preview watcher, refactor cube cloud package #2418

Merged
merged 25 commits into from
Mar 25, 2021

Conversation

RusovDmitriy
Copy link
Member

Check List

  • Tests has been run in packages where changes made if available
  • Linter has been run for changed code
  • Tests for the changes have been added if not covered yet
  • Docs have been added / updated if required

Issue Reference this PR resolves

[For example #12]

Description of Changes Made (if issue reference is not provided)

[Description goes here]

@codecov
Copy link

codecov bot commented Mar 23, 2021

Codecov Report

Merging #2418 (67a937d) into master (a2cde27) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2418   +/-   ##
=======================================
  Coverage   56.35%   56.35%           
=======================================
  Files         118      118           
  Lines        8714     8714           
  Branches     1887     1887           
=======================================
  Hits         4911     4911           
  Misses       3437     3437           
  Partials      366      366           

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 720fec7...67a937d. Read the comment docs.

@RusovDmitriy RusovDmitriy changed the title feat: Live preview mode feat: Live preview watcher Mar 23, 2021
@RusovDmitriy RusovDmitriy marked this pull request as ready for review March 23, 2021 08:18
@RusovDmitriy RusovDmitriy requested a review from a team as a code owner March 23, 2021 08:18
@RusovDmitriy RusovDmitriy requested a review from ovr March 23, 2021 08:18
if (this.hooks.onStart) this.hooks.onStart(files);

try {
for (let i = 0; i < files.length; i++) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for (const file of files) m?)


if (!upstreamHashes[filePosix] || upstreamHashes[filePosix].hash !== fileHashes[file].hash) {
if (this.hooks.onUpload) this.hooks.onUpload(files, file);
await this.cubeCloudClient.uploadFile({
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe it's better to use await Promise.all() WDYT?

@RusovDmitriy RusovDmitriy changed the title feat: Live preview watcher feat: Live preview watcher, refactor cube cloud package Mar 24, 2021
@RusovDmitriy RusovDmitriy merged commit a311843 into master Mar 25, 2021
@RusovDmitriy RusovDmitriy deleted the feat-live-preview-watcher branch March 25, 2021 07:11
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

2 participants