Skip to content

Allow dependencies to upgrade with non-breaking changes#49

Merged
jsloyer merged 2 commits into
IBM-Cloud:masterfrom
adrukh:master
Jan 18, 2019
Merged

Allow dependencies to upgrade with non-breaking changes#49
jsloyer merged 2 commits into
IBM-Cloud:masterfrom
adrukh:master

Conversation

@adrukh

@adrukh adrukh commented Dec 26, 2018

Copy link
Copy Markdown

Following a chat on #46 (comment), applying ^ to production dependencies' versions to remove vulnerabilities introduced by dependencies.

Before: http://snyk.io/test/github/IBM-Cloud/cf-nodejs-client
After: http://snyk.io/test/github/adrukh/cf-nodejs-client

Using the `^` notation to allow the deps of this package to move forward as long as non-breaking changes are introduced to the dependencies.

This will remove 9 out 10 vulnerabilities currently introduced by this package's dependencies.
Allow a manual release with a newer version.
@adrukh

adrukh commented Dec 26, 2018

Copy link
Copy Markdown
Author

Travis tests seem to only enforce linting on PRs, I wonder how can the functional correctness be validated. Could not run npm run test:local locally, assume it depends on a CF setup I do not have?

@adrukh

adrukh commented Dec 31, 2018

Copy link
Copy Markdown
Author

👋 @srl295 how do you suggest to proceed?

@adrukh

adrukh commented Jan 7, 2019

Copy link
Copy Markdown
Author

👋 @srl295 :)

@adrukh

adrukh commented Jan 18, 2019

Copy link
Copy Markdown
Author

👋 @srl295 :))

@jsloyer jsloyer merged commit fde6ca3 into IBM-Cloud:master Jan 18, 2019
@adrukh

adrukh commented Jan 20, 2019

Copy link
Copy Markdown
Author

THANKS!

@adrukh

adrukh commented Jan 20, 2019

Copy link
Copy Markdown
Author

Can you please update once https://www.npmjs.com/package/cf-client is updated with this release? 🙏

@adrukh

adrukh commented Jan 28, 2019

Copy link
Copy Markdown
Author

👋 @jsloyer :)

@adrukh

adrukh commented Mar 12, 2019

Copy link
Copy Markdown
Author

👋 @jsloyer will keep poking here regarding publishing a new release #sorrynotsorry :)

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