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

Update bbs.js to for Config Init Enhancements #574

Merged
merged 2 commits into from
Nov 21, 2024

Conversation

crhultay
Copy link
Contributor

If a new user of enigma-bbs tries to run ./main.js it will now explain why it can't start and suggest running './oputil.js config new'

In addition, in the event of an error scenario it will exit with exitcode 1 so that systemd / other tools that bring up the service will be able to detect than an error had occurred -- process.exit() defaults to 0, and therefore would not have been caught.

If a new user of enigma-bbs tries to run ./main.js it will now explain why it can't start and suggest running './oputil.js config new'

In addition, in the event of an error scenario it will exit with exitcode 1 so that systemd / other tools that bring up the service will be able to detect than an error had occurred -- process.exit() defaults to 0, and therefore would not have been caught.
core/bbs.js Outdated Show resolved Hide resolved
Copy link
Owner

@NuSkooler NuSkooler left a comment

Choose a reason for hiding this comment

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

Thank you again, very much appreciated!

@NuSkooler NuSkooler merged commit dd30631 into NuSkooler:master Nov 21, 2024
2 checks passed
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.

None yet

2 participants