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

nodejs-loopback: upgrade/add application dependencies #488

Merged
merged 2 commits into from
Nov 21, 2019

Conversation

raymondfeng
Copy link
Contributor

@raymondfeng raymondfeng commented Nov 14, 2019

To develop the application independent of the base image, we need to set up
dependencies in application's package.json file.

Checklist:

Modifying an existing stack:

  • Updated the stack version in stack.yaml

Contributing a new stack:

  • Describe how application dependencies are managed:

  • Explain how Appsody file watcher is utilized:

  • Describe other Appsody environment variables defined by the stack image:

  • Describe any limitations and known issues:

Related Issues:

Copy link
Contributor

@neeraj-laad neeraj-laad left a comment

Choose a reason for hiding this comment

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

@raymondfeng Overall looks good. It seems like we have added metrics support in the stack. I believe we should update the README.md to reflect that capability is available from the stack.

To develop the application independent of the base image, we need to set up
dependencies in application's package.json file.
@raymondfeng
Copy link
Contributor Author

@neeraj-laad README updated. Thanks for the review.

Copy link
Collaborator

@Kamran64 Kamran64 left a comment

Choose a reason for hiding this comment

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

LGTM - @raymondfeng I checked this with Neeraj too.

Copy link
Contributor

@neeraj-laad neeraj-laad left a comment

Choose a reason for hiding this comment

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

LGTM

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.

3 participants