Not sure if this is the right spot for this but it would be really helpful if the aws BatchSubmit task printed the error message on this line rather than just catching it
n
nicholas
07/13/2021, 8:49 PM
Hi @Kathryn Klarich - tasks in the task library are typically community-contributed so we welcome PRs on those for any improvements you see 🙂
k
Kathryn Klarich
07/13/2021, 8:49 PM
yes i just remembered that, i'll look into it now, thanks!
😄 1
a
Amanda Wee
07/13/2021, 8:53 PM
Just wondering: doesn't the exception chaining mean that the error message is still in the logs, so printing it at that point would be unnecessary?
k
Kathryn Klarich
07/14/2021, 3:04 PM
No, the error message doesn't show up in the logs, at least not anywhere that i can find - let me know if you are able to see it somewhere though @Amanda Wee!
Kathryn Klarich
07/14/2021, 3:08 PM
@nicholas I am trying to contribute my change (PR here) but I can't figure out how to add a file to the changes directory - I wanted to use the PR # as the filename, which is why i submitted the PR first (so that the number could be assigned), but then I'm not sure how to upload my file.
n
nicholas
07/14/2021, 3:10 PM
Hi @Kathryn Klarich - have you looked at the changelog entries readme? I think that'll help but the TLDR is you'll want to create the file on your branch and commit it as you would any other file