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

Update all dependencies #257

Merged
merged 5 commits into from
Jul 26, 2018
Merged

Update all dependencies #257

merged 5 commits into from
Jul 26, 2018

Conversation

rejas
Copy link
Contributor

@rejas rejas commented Jul 16, 2018

No description provided.

@sjentzsch
Copy link

Would love to see dependency version being upgraded. Does anyone has an idea why the build fail? The job logs (to me at least) do not tell much.

@rejas
Copy link
Contributor Author

rejas commented Jul 23, 2018

Narrowed the build error down to the coffeescript dependency. I am not to familiar with it to fix that.
The rest of the updated dependencies at least removes all warnings from npm audit fix.

@sjentzsch sjentzsch mentioned this pull request Jul 25, 2018
@avevlad avevlad merged commit a80e3e5 into avevlad:master Jul 26, 2018
@rejas rejas deleted the update_dependencies branch July 26, 2018 20:23
@rejas
Copy link
Contributor Author

rejas commented Jul 27, 2018

Thx for merging @avevlad Any estimation when you will release a new version to npm?

@avevlad
Copy link
Owner

avevlad commented Aug 28, 2018

@rejas [email protected]

[email protected] prepublish .
coffee -o . -bc src/

npm notice
npm notice 📦 [email protected]
npm notice === Tarball Contents ===
npm notice 1.4kB package.json
npm notice 72B .travis.yml
npm notice 8.7kB index.js
npm notice 167.8kB jetbrains.png
npm notice 1.1kB LICENSE
npm notice 4.6kB README.md
npm notice 768B bin/make_certs.sh
npm notice 1.1kB certs/server.crt
npm notice 1.7kB certs/server.key
npm notice 6.2kB src/index.coffee
npm notice 8B test/fixtures/multiple/app/index.html
npm notice 9B test/fixtures/multiple/dist/dist.html
npm notice 11B test/fixtures/simplest/index.html
npm notice 11B test/fixtures/simplest/test.txt
npm notice 6.0kB test/test.js
npm notice === Tarball Details ===
npm notice name: gulp-connect
npm notice version: 5.6.1
npm notice package size: 162.1 kB
npm notice unpacked size: 199.4 kB

@avevlad
Copy link
Owner

avevlad commented Aug 28, 2018

@rejas Thanks!

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.

3 participants