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

Bug - Issue with github comment on resolved items #1082

Closed
juancarlosjr97 opened this issue Feb 18, 2024 · 10 comments
Closed

Bug - Issue with github comment on resolved items #1082

juancarlosjr97 opened this issue Feb 18, 2024 · 10 comments

Comments

@juancarlosjr97
Copy link
Contributor

v17.1.0 brings the functionality to add comments on PRs in CI environments.

However, it appears to be an issue with the feature as it now fails when trying to add a comment. See here.

The release does not fail as it happened before but execution fails.

@juancarlosjr97
Copy link
Contributor Author

juancarlosjr97 commented Feb 18, 2024

I have not had time to look into it, just noticed it one a release using the latest version.

@juancarlosjr97
Copy link
Contributor Author

The issue is...

file:///home/runner/work/release-it-pre-releases/release-it-pre-releases/node_modules/ora/index.js:385
		throw new TypeError('Parameter `action` must be a Function or a Promise');
		      ^

TypeError: Parameter `action` must be a Function or a Promise

From here https://github.com/sindresorhus/ora/blob/main/index.js#L384-L385

@webpro
Copy link
Collaborator

webpro commented Feb 18, 2024

If it's an issue with Ora then there might be something off with --ci or CI env var not set? Otherwise you can try to set github.comments.submit to false.

Of course there can be a bug in release-it, too.

@juancarlosjr97
Copy link
Contributor Author

The action is running npx release-it --ci --git.tagExclude='*[-]*'.

@juancarlosjr97
Copy link
Contributor Author

Just in case, I tried updating the release configuration to github comments to false. See here and the commit

@juancarlosjr97
Copy link
Contributor Author

Well, it is correctly going to that scope as the --ci flag is set. Now, I am trying to understand why the this.commentOnResolvedItems() is identified as not a Promise.

@webpro
Copy link
Collaborator

webpro commented Feb 18, 2024

🚀 This issue has been resolved in v17.1.1. See Release 17.1.1 for release notes.

@webpro webpro closed this as completed in b46d5bd Feb 18, 2024
@webpro
Copy link
Collaborator

webpro commented Feb 18, 2024

Maybe fixed it, shot in the dark 🔫

@juancarlosjr97
Copy link
Contributor Author

I got green pipeline across projects :)

Thank you!

@webpro
Copy link
Collaborator

webpro commented Feb 18, 2024

Green is good! piew piew 🔫

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

No branches or pull requests

2 participants