Skip to content
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: better error logging for network errors #245

Merged
merged 2 commits into from
Jun 22, 2021

Conversation

nfriend
Copy link
Contributor

@nfriend nfriend commented Jun 17, 2021

What does this PR do?

Adds some additional logging to publish.js, including error logs when requests to GitLab's API fail.

Why?

Currently, if a request to GitLab's API fails, this plugin logs a generic error message without any specifics. This makes it hard for end-users to diagnose and fix issues.

@nfriend
Copy link
Contributor Author

nfriend commented Jun 17, 2021

@keplersj I see you're one of the other maintainers of this plugin 😄 Would you mind reviewing this PR?

cc @travi

Copy link
Member

@travi travi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems reasonable to me 👍🏼

@nfriend
Copy link
Contributor Author

nfriend commented Jun 22, 2021

Proceeding with a self-merge since this has @travi's approval above.

(Please let me know if this isn't the proper process!)

@nfriend nfriend merged commit 25a9011 into semantic-release:master Jun 22, 2021
@github-actions
Copy link

🎉 This PR is included in version 6.2.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@travi
Copy link
Member

travi commented Jun 23, 2021

(Please let me know if this isn't the proper process!)

you're good to merge as long as nothing is preventing it. i usually wont merge when you have the premission since you have more context on whether you are wanting to wait for anything else before it happens. thanks for your help here :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants