Suggested language acknowledging shelljs as a reasonable "production"…#99
Open
boutell wants to merge 1 commit intoForbesLindesay:masterfrom
Open
Suggested language acknowledging shelljs as a reasonable "production"…#99boutell wants to merge 1 commit intoForbesLindesay:masterfrom
boutell wants to merge 1 commit intoForbesLindesay:masterfrom
Conversation
… application of the module
Author
|
Thanks. |
|
However it does make sense to use this in many more kinds of production cases, including REST services and whatnot. Not all code in a service is involved in handling the service requests. For example think about loading configuration data (such as signing secrets and keys) from a configuration server at startup. A very reasonable use case that can be relevant to any kind of production app. Doing this synchronously can greatly simplify initialization that would otherwise have to be delayed using other mechanisms and which sometimes even includes loading of dependencies while causing minimal or no impact to performance or resource utilization. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
… application of the module