Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggested language acknowledging shelljs as a reasonable "production"… #99

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

boutell
Copy link

@boutell boutell commented Jun 18, 2018

… application of the module

@boutell
Copy link
Author

boutell commented Jun 18, 2018

Thanks.

@dreusel
Copy link

dreusel commented Feb 28, 2023

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants