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

remove godep in favor of gpm #458

Merged
merged 4 commits into from
Sep 14, 2014
Merged

remove godep in favor of gpm #458

merged 4 commits into from
Sep 14, 2014

Conversation

mreiferson
Copy link
Member

gpm is exactly the kind of lightweight tool we're looking for in order to automate reproducible builds from a list of dependencies.

Unfortunately godep went in a different (valid) direction.

@mreiferson
Copy link
Member Author

RFR @jehiah

@jphines
Copy link
Contributor

jphines commented Sep 14, 2014

👍

}
]
}
code.google.com/p/snappy-go/snappy 12e4b4183793ac4b061921e7980845e750679fd0
Copy link
Member

Choose a reason for hiding this comment

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

can we comment human readable versions on these lines (where we have such a thing) or the date they were last bumped or something? (looks like anything after # should be treated as a comment)

Copy link
Member Author

Choose a reason for hiding this comment

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

in terms of date of bumps, that's all in the git history

Copy link
Member

Choose a reason for hiding this comment

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

fair

@jehiah
Copy link
Member

jehiah commented Sep 14, 2014

LGTM aside from that small detail

@mreiferson
Copy link
Member Author

updated and squashed in

gonna separately bump some deps after this and #404 lands

jehiah added a commit that referenced this pull request Sep 14, 2014
@jehiah jehiah merged commit 23a4b88 into nsqio:master Sep 14, 2014
@mreiferson mreiferson deleted the bye_godep_458 branch September 14, 2014 21:14
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.

3 participants