-
Notifications
You must be signed in to change notification settings - Fork 137
PRs are no longer resolved with closed/fixed keywords #942
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
Comments
seems also related to semantic-release/semantic-release#3476 @babblebey could this be related to a recent change? |
Yes @travi, looks related to an error from the GraphQL call to get details for resolved issue in PR body. Quick question @wolfy1339, is the entity with number I ask this because I think this might be some edge case I've left unaddressed, i.e. if Thinking out loud 🤔 |
That would be a PR |
Then, that's the edge case! Should patch a fix for this in a bit 🤌🏾 The solution would be to try ignore empty responses from parsed issue number in PR body/Commits messages, this because not all these numbers belong to an issue, some are PRs (like in your case where a PR is linked to be closed by another PR). Got any thoughts on this!??? 🤔 |
I just tried to release my package and got this error https://github.com/AllanOricil/nrg/actions/runs/11545022673/job/32131354251 2 Weeks ago it was working with no issues |
🎉 This issue has been resolved in version 11.0.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
I'm adding this here since it seems related but it could have completely different root cause though. Does anyone have an idea? |
It seems that something has broken and semantic release is unable to resolve PRs in commit bodies.
It used to do this without any issue at all.
There isn't any custom plugins used, it's the default configuration for semantic-release
https://github.com/probot/pino/actions/runs/11371733573/job/31634467759?pr=321
The text was updated successfully, but these errors were encountered: