Skip to content

Comments

Partial refactor, multi region, version tag#2

Merged
elbuo8 merged 10 commits intomasterfrom
multi-deploy
Feb 3, 2015
Merged

Partial refactor, multi region, version tag#2
elbuo8 merged 10 commits intomasterfrom
multi-deploy

Conversation

@elbuo8
Copy link
Contributor

@elbuo8 elbuo8 commented Feb 3, 2015

No description provided.

@elbuo8
Copy link
Contributor Author

elbuo8 commented Feb 3, 2015

@motdotla why is there a test for the package.json version? Shouldn't it just test for existence?

@motdotla
Copy link
Owner

motdotla commented Feb 3, 2015

I like to test for package version, because it forces me to make sure I do bump the version. It acts as sort of a second check.

Copy link
Owner

Choose a reason for hiding this comment

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

nice. airbnb style.

@motdotla
Copy link
Owner

motdotla commented Feb 3, 2015

👍

@elbuo8
Copy link
Contributor Author

elbuo8 commented Feb 3, 2015

Last push makes sure we stick to single quotes.

elbuo8 pushed a commit that referenced this pull request Feb 3, 2015
Partial refactor, multi region, version tag
@elbuo8 elbuo8 merged commit 0caa7c6 into master Feb 3, 2015
DeviaVir pushed a commit that referenced this pull request Apr 9, 2016
Merge upstream/master with kecaps:master
@DeviaVir DeviaVir deleted the multi-deploy branch May 12, 2016 09:48
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