Skip to content

Conversation

etpinard
Copy link
Contributor

cc @plotly/plotly_js

Copy link
Contributor

@archmoj archmoj left a comment

Choose a reason for hiding this comment

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

Thanks @etpinard.
I simply noticed few places where the modules were were resolved to http instead on https.

@etpinard
Copy link
Contributor Author

Good eye @archmoj -> 31066fa

"gzip-size": "^5.0.0",
"image-size": "^0.6.3",
"into-stream": "^3.1.0",
"into-stream": "^4.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

FYI - Now supports TypedArray and ArrayBuffer sindresorhus/into-stream@ec6ef5c

"node-sass": "^4.10.0",
"npm-link-check": "^2.0.0",
"node-sass": "^4.11.0",
"npm-link-check": "^3.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

Cool module by @etpinard

@archmoj
Copy link
Contributor

archmoj commented Dec 22, 2018

Nicely done! 💃
It would be great if we could follow up the cwise PR #3267 for the next version.

@etpinard etpinard merged commit 340ea44 into master Dec 24, 2018
@etpinard etpinard deleted the bump-dev-dep-2018-12-21 branch December 24, 2018 14:56
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