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

Extract config into its own package #1976

Merged
merged 4 commits into from
Aug 5, 2015
Merged

Extract config into its own package #1976

merged 4 commits into from
Aug 5, 2015

Conversation

tamird
Copy link
Contributor

@tamird tamird commented Aug 5, 2015

This does a bit of cleanup towards #629.

@tbg tbg added the PTAL label Aug 5, 2015
@tamird
Copy link
Contributor Author

tamird commented Aug 5, 2015

The next steps here would be:

@bdarnell
Copy link
Contributor

bdarnell commented Aug 5, 2015

LGTM

tamird added a commit that referenced this pull request Aug 5, 2015
Extract config into its own package
@tamird tamird merged commit 29b0d09 into cockroachdb:master Aug 5, 2015
@tbg tbg removed the PTAL label Aug 5, 2015
@tamird tamird deleted the config-package branch August 5, 2015 19:46
mrtracy pushed a commit to mrtracy/cockroach that referenced this pull request Aug 6, 2015
The field name of this structure was changed by cockroachdb#1976, but it has since been
decided that "address" is a more accurate name than "string".
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.

3 participants