<@U06CBSQNLP5> - Any thoughts on your comment here...
# contributing
a
@julian_knight - Any thoughts on your comment here?: https://gitlab.com/meltano/meltano/-/merge_requests/2221#note_648001137 @edgar_ramirez_mondragon and I weren't clear if you see this as an issue. Also, I could not tell if this were new (regression) or already-existing condition (same as-is).
j
@aaronsteers I saw that as an issue with
raise KeyboardInterrupt()
so I changed it to instead of raising propigate the signal and otherwise ignore it the first time. This addresses the Docker race concern raised in my note. I tested it locally and it worked as expected so I think that thread can be closed, unless y'all have any concerns about the behavior
a
Looks good! Thanks for the clarification. 👍 cc @edgar_ramirez_mondragon
e
Yup, looks good. Good to merge from me