search

strongloop / strong-deploy

Deploy nodejs applications
Other
13 stars 3 forks source link

Support multiapp pm in deploy APIs #41

Closed sam-github closed 9 years ago

sam-github commented 9 years ago

sl-deploy was resolving a service name to the deploy URL, but this is commonly needed code: the deploy APIs can't be used if you don't know the URl to deploy to, and its the mesh client that can tell us. Refactor so the APIs do the resolution.

In the process, fix local deploy, which didn't get multiapp support when it was added for packfile and git deployments.

connected to strongloop/strong-start#8

rmg commented 9 years ago

Could possibly use a unit test for the extracted function... but I don't care that much.

LGTM.