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

FIX for image-snapshot being broken && FIX snapshots #5457

Closed
wants to merge 2 commits into from

Conversation

ndelangen
Copy link
Member

by actually serving content (no file://)

@ndelangen ndelangen added the maintenance User-facing maintenance tasks label Feb 1, 2019
@ndelangen ndelangen added this to the v5.0.0 milestone Feb 1, 2019
@ndelangen ndelangen self-assigned this Feb 1, 2019
@ndelangen ndelangen force-pushed the tech/fix-image-snapshot branch from a174ac0 to 5dd4f0d Compare February 1, 2019 23:49
@ndelangen ndelangen force-pushed the tech/fix-image-snapshot branch from 5dd4f0d to e64dfac Compare February 2, 2019 00:19
@ndelangen ndelangen requested a review from igor-dv as a code owner February 2, 2019 02:12
@tmeasday tmeasday force-pushed the tech/fix-image-snapshot branch from ad8b053 to e64dfac Compare February 2, 2019 02:15
@codecov
Copy link

codecov bot commented Feb 2, 2019

Codecov Report

Merging #5457 into next will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##             next    #5457   +/-   ##
=======================================
  Coverage   31.38%   31.38%           
=======================================
  Files         635      635           
  Lines        9190     9190           
  Branches     1271     1292   +21     
=======================================
  Hits         2884     2884           
  Misses       5666     5666           
  Partials      640      640
Impacted Files Coverage Δ
lib/components/src/preview/preview.js 0% <0%> (ø) ⬆️

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 d3b65e9...c96b51b. Read the comment docs.

@tmeasday
Copy link
Member

tmeasday commented Feb 4, 2019

@ndelangen could be a memory issue but here is what chrome logs:

https://circleci.com/gh/storybooks/storybook/95407

Googling implied maybe it was [email protected] but downgrading to 1.11 doesn't seem to have helped.

@ndelangen ndelangen closed this Feb 9, 2019
@ndelangen ndelangen deleted the tech/fix-image-snapshot branch May 7, 2019 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance User-facing maintenance tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants