Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

chore: rename package #79

Merged
merged 3 commits into from
Jul 2, 2018
Merged

chore: rename package #79

merged 3 commits into from
Jul 2, 2018

Conversation

vasco-santos
Copy link
Member

@vasco-santos vasco-santos commented Jun 30, 2018

I think that it makes sense to reset the package version. But, I don't know the usual procedure here. This way, I can also keep the same version if you usually do it this way.

Once this is merged, tasks that need to be done:

  • deprecate libp2p-railing
  • publish minor version of libp2p-bootstrap
  • change github repo name

Commands:
npm deprecate [email protected] "WARNING: This project has been renamed to libp2p-bootstrap. Install using libp2p-bootstrap instead."

Closes #67

@ghost ghost assigned vasco-santos Jun 30, 2018
@ghost ghost added the status/in-progress In progress label Jun 30, 2018
Copy link
Member

@daviddias daviddias left a comment

Choose a reason for hiding this comment

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

You will need to change the github urls on the package.json as well.

@vasco-santos
Copy link
Member Author

Yeah, forgot the urls, thanks!

Changed now.

package.json Outdated
@@ -1,6 +1,6 @@
{
"name": "libp2p-railing",
"version": "0.9.2",
Copy link
Member

Choose a reason for hiding this comment

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

preserve the version

@vasco-santos
Copy link
Member Author

vasco-santos commented Jun 30, 2018

@diasdavid version is kept now. After this being merged, and the npm packages being deprecated and published, I need you to change the repo name (task added in the first message)

EDIT: it seems that you already renamed it. Thanks 🙂

@vasco-santos vasco-santos merged commit 2bef429 into master Jul 2, 2018
@ghost ghost removed the status/in-progress In progress label Jul 2, 2018
@vasco-santos vasco-santos deleted the chore/rename-package branch July 2, 2018 09:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants