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

Bump aws-sdk from 2.127.0 to 2.134.0 #144

Closed
wants to merge 1 commit into from

Conversation

dependabot-preview[bot]
Copy link
Contributor

Bumps aws-sdk from 2.127.0 to 2.134.0.

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase?.

If you'd like to skip this version, you can just close this PR. If you have any feedback just mention @dependabot in the comments below.

@bfirsh
Copy link
Member

bfirsh commented Oct 17, 2017

@dependabot - My yarn.lock contains two versions of aws-sdk. One which is a direct dependency, one which is a dependency of a dependency. You've just upgraded the direct dependency, even though the dependency of the dependency can also be upgraded.

This is the default behaviour of yarn upgrade aws-sdk sadly, but it'd be cool if you could do something clever. :)

@dependabot-preview
Copy link
Contributor Author

Interesting, thanks for the feedback - we’ll have a think. Are there any cases where you wouldn’t want that behaviour?

(I think you have to @dependabot on any reply - I’ll change that, but for now it’s the case.)

@bfirsh
Copy link
Member

bfirsh commented Oct 17, 2017

@dependabot I'm not sure. I assumed yarn would work that way by default, but because it doesn't, perhaps there are subtleties I'm missing. If the sub-dependency is specified in a way that means it's compatible with the newer version, and the tests pass, I see no reason not to include that change.

I can envisage having different versions all over the place causing more problems than the risk of upgrading dependencies (e.g. incompatible objects being passed around, multiple singletons, hard-to-track down bugs, etc).

See here for more discussion: yarnpkg/yarn#2394

@dependabot-preview
Copy link
Contributor Author

Superseded by #145.

@dependabot-preview dependabot-preview bot deleted the dependabot/yarn/aws-sdk-2.134.0 branch October 18, 2017 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant