-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add options for env, startedBy and region and add a trap for SIGINT #1
Conversation
… and add --region, --started-by and --env
Add new options and SIGINT handling
Hi Johannes, Thank you for your contributions! Sorry I didn't see this PR earlier. At a quick glance there's only one non style related change i'd like to see, and i'll comment on the specify line re that change. Style wise I'm not a fan of multi line variable declarations with a single Thanks, |
index.js
Outdated
var stream = combiner(logs, formatter); | ||
stream.logStream = logs; | ||
|
||
process.on('SIGINT', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Listening for the interrupt signal should probably happen in the ./bin/ecs-task-runner
script. Since this index.js
is intended to be used for module use.
Hi Coen,
Sorry for my late reply and thank you for getting back to me. I think your comments make perfect sense and I’m happy to adjust the PR.
I’ll try to get to it asap. Will ping you once it is updated.
Thanks
Johannes Vetter | Lead Engineer
johannes@art19.com <mailto:johannes@art19.com>
ART19, Inc. • 13261 Moorpark St Suite 104 • Sherman Oaks CA 91423 • 866.882.7819 <tel://415.423.3583> • www.art19.com <http://www.art19.com/>
This email and any attachments thereto may contain private, confidential, and privileged material for the sole use of the intended recipient. Any review, copying, or distribution of this email (or any attachments thereto) by others is strictly prohibited. If you are not the intended recipient, please contact the sender immediately and permanently delete the original and any copies of this email and any attachments thereto.
… Am 12.01.2018 um 14:40 schrieb Coen Hyde ***@***.***>:
Hi Johannes,
Thank you for your contributions! Sorry I didn't see this PR earlier. At a quick glance there's only one non style related change i'd like to see, and i'll comment on the specify line re that change. Style wise I'm not a fan of multi line variable declarations with a single var or const. Can we update the ESLint rules to enforce const variable=value; style?
Thanks,
Coen
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#1 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABGBDU19Vp6k-WGeDRp_Q9ylRVWAR2Wsks5tJ99cgaJpZM4RSYTp>.
|
Hey @johvet, sorry I missed this. These are some good changes. Merging and releasing a new version |
No description provided.