Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

Run as a Background Process #249

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

FinnLawrence
Copy link

Added new CLI option flags:

  • --background, aliased to -b. Runs the server using a Daemon process in the background.
  • --output, sets the output location for the logger. Only used if --background is true, and the quiet mode flag has not been set.

I needed this to be able to run the FakeS3 server and my test suite easily in our docker container.

  • Tests passing.
  • CLA executed.

@limcross
Copy link

First of all, this is a a fantastic new feature 👍 so thanks for your time @FinnLawrence. I hope the contributing team will see this soon.

Without prejudice of the previous, instead of -b (background), could be great use -d (daemon), i think is more conventional. Tell me what you think about it

@FinnLawrence
Copy link
Author

First of all, this is a a fantastic new feature 👍 so thanks for your time @FinnLawrence. I hope the contributing team will see this soon.

Without prejudice of the previous, instead of -b (background), could be great use -d (daemon), i think is more conventional. Tell me what you think about it

Good suggestion @limcross, I've amended the flag and descriptions.

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.

None yet

2 participants