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 devDeps, fix lint error #306

Merged
merged 1 commit into from Oct 30, 2016
Merged

Update devDeps, fix lint error #306

merged 1 commit into from Oct 30, 2016

Conversation

RyanZim
Copy link
Collaborator

@RyanZim RyanZim commented Oct 29, 2016

@jprichardson Ever think of using @greenkeeperio-bot so I don't have to do this? I use it on all of my projects and really like it. Not only does it update dependencies outside your version range, it also checks to make sure none of your dependencies break semver.

My only gripe is the commit messages: chore(package): update eslint to version 3.8.0. I always use the squash-merge function so I can reword the commit message to Update eslint to version 3.8.0.

https://greenkeeper.io/

@jprichardson jprichardson merged commit 9d19da7 into master Oct 30, 2016
@jprichardson
Copy link
Owner

Thanks for this!

Ever think of using @greenkeeperio-bot so I don't have to do this?

I'm not a fan of it as I think that this promotes updating dependencies for the sake of updating dependencies as a good thing. This is not to say that I couldn't warm up to it, but I'm not ready for it yet.

@RyanZim RyanZim deleted the deps branch October 31, 2016 14:59
@RyanZim
Copy link
Collaborator Author

RyanZim commented Oct 31, 2016

I think that this promotes updating dependencies for the sake of updating dependencies as a good thing

Not sure what you mean by that, it only updates your deps when the latest version is not covered by your current version range. Mind elaborating?

@jprichardson
Copy link
Owner

jprichardson commented Oct 31, 2016

Not sure what you mean by that, it only updates your deps when the latest version is not covered by your current version range. Mind elaborating?

Sure. There's a unforeseen cost when updating dependencies. You rarely don't know what it is up front. Greenkeeper and the like promote that to have stable and usable software, you must update your dependencies. Software developers feel good about this - one more thing to check off the list. I'm saying that these services don't show the unforeseen cost. I believe it's the developer duty and responsibility to be judicious about updating and the need to update.

Please feel free to ask more questions about this if you have them.

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.

None yet

2 participants