Skip to content
This repository has been archived by the owner on Sep 14, 2022. It is now read-only.

Deprecate swagger-node-express, import swagger-node #207

Merged
merged 330 commits into from
Jun 3, 2015
Merged

Deprecate swagger-node-express, import swagger-node #207

merged 330 commits into from
Jun 3, 2015

Conversation

theganyo
Copy link
Contributor

@theganyo theganyo commented May 5, 2015

This PR deletes all existing files in swagger-node-express and imports all the files from swagger-node, maintaining history for both. The new project is a CLI that creates and manages connect, express, sails, hapi, and restify projects using Swagger 2.0. This project is intended to be published to npmjs as "swagger" - replacing the existing "swagger-node" project there that will be deprecated and removed.

@theganyo
Copy link
Contributor Author

theganyo commented Jun 3, 2015

Thanks, @fehguy!

@DaniloOliveira28
Copy link

It looks like a great initiative. Ty @theganyo and @fehguy for the work!

However I am feeling lost with all these changes, I was using the old swagger-node-express in my projects. What repo I need to follow? I found three with similar code:

https://github.com/apigee-127/swagger-node
https://github.com/apigee-127/swagger-express
https://github.com/swagger-api/swagger-node

What is the objective of each one?

Thanks

@theganyo
Copy link
Contributor Author

theganyo commented Jun 6, 2015

Sorry it's confusing. We clearly need to do a better job of explaining how the pieces fit together.

But to answer your question:

The first repo will be going away with the move.

The second provides middleware to a swagger-node app using express.

The third is the primary repo for the swagger-node toolkit now. It is the CLI that creates and manages project and provides other tools for Swagger such as validating a specification doc.

@webron
Copy link
Contributor

webron commented Jun 6, 2015

@theganyo - if this one replaces the one at a127, perhaps until that one is removed, add a clear note at the top of the project's README that it has moved here?

@theganyo
Copy link
Contributor Author

theganyo commented Jun 7, 2015

Done!

@prabhatjha
Copy link
Contributor

I think we should also delete the swagger-node repo from apigee-127 org.
WDYT?

On Sun, Jun 7, 2015 at 10:34 AM, Scott Ganyo notifications@github.com
wrote:

Done!


Reply to this email directly or view it on GitHub
#207 (comment)
.

Cheers,

Prabhat

Help spread www.eejot.org

@theganyo
Copy link
Contributor Author

theganyo commented Jun 7, 2015

Yes, that's what I said above. However, until we release a new version to npmjs, we still have a link that points at the apigee-127. This will be very soon, but not now.

@webron
Copy link
Contributor

webron commented Jun 7, 2015

Since we didn't do a project transfer, but merge the code, I'd suggested not deleting the repo for a few months but emptying it from content, leaving a readme explaining the project have moved here and put a link.

This would allow for previous users to follow the link more easily.

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.

None yet