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
fix(fail): properly parse issue creation response to log in terminal #680
fix(fail): properly parse issue creation response to log in terminal #680
Conversation
Thanks @JonasSchubert, the changes looks good. Can you please fix the email/signature in your commit so that GitHub is happy? 🙇 |
🙈 sorry. work email and private open source email. Will fix that later today or tomorrow latest. |
@fgreinacher Updated email and gpg. Thanks for realising that 🙇 |
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.
Thanks @JonasSchubert, had a look at the changes again and maybe we can simplify the test a bit :)
9224517
to
4014f51
Compare
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.
Perfect, thanks for the fix!
🎉 This PR is included in version 13.0.3 🎉 The release is available on: Your semantic-release bot 📦🚀 |
I realized in our logs, the
id
andweb_url
are not parsed properly. According to the docs the call to the method.json()
was missing. Local tests showed this fixed the logs and did not affect the normal functionality.