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

config: rename addresses #121

Merged
merged 2 commits into from
Sep 26, 2014
Merged

config: rename addresses #121

merged 2 commits into from
Sep 26, 2014

Conversation

jbenet
Copy link
Member

@jbenet jbenet commented Sep 24, 2014

WARNING: change breaks old configs.
@whyrusleeping @perfmode

This commit changes the way addresses are stored in config files. It lumps
Identity.Address and RPCAddress into Addresses. This commit also fixes several
golint issues.

WARNING: change breaks old configs.
@whyrusleeping @perfmode

This commit changes the way addresses are stored in config files.
It lumps Identity.Address and RPCAddress into Addresses. This
commit also fixes several golint issues.
@jbenet jbenet added the status/in-progress In progress label Sep 24, 2014
@jbenet jbenet mentioned this pull request Sep 24, 2014
@btc
Copy link
Contributor

btc commented Sep 24, 2014

LGTM

@btc btc added codereview and removed status/in-progress In progress labels Sep 24, 2014
@jbenet
Copy link
Member Author

jbenet commented Sep 26, 2014

Merging. -- watch out folks, will break your configs. (perhaps overkill but...) i wrote a script to help your migrations: https://gist.github.com/jbenet/0ae83df1a0d4e5d8d37b -- maybe in the future it should be automatic.

jbenet added a commit that referenced this pull request Sep 26, 2014
@jbenet jbenet merged commit 80a927f into master Sep 26, 2014
@jbenet jbenet removed the codereview label Sep 26, 2014
@jbenet jbenet deleted the feat/addrs-in-config branch September 26, 2014 09:36
gammazero added a commit that referenced this pull request May 4, 2021
@aschmahmann aschmahmann mentioned this pull request May 14, 2021
71 tasks
laurentsenta pushed a commit to laurentsenta/kubo that referenced this pull request Feb 25, 2022
remove duplicate entries in defaultServerFilters
laurentsenta pushed a commit to laurentsenta/kubo that referenced this pull request Feb 25, 2022
remove duplicate entries in defaultServerFilters
laurentsenta pushed a commit to laurentsenta/kubo that referenced this pull request Mar 4, 2022
remove duplicate entries in defaultServerFilters
laurentsenta pushed a commit to laurentsenta/kubo that referenced this pull request Mar 4, 2022
remove duplicate entries in defaultServerFilters
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