Changelog workflow is broken

[andrewfullard]

Describe the bug

Changelog workflow is refusing to push, claiming that it hasn't got a branch checked out.

To Reproduce

Run changelog workflow

Screenshots

[madoyi]

I'd like to help out with this issue, is there a specific implementation you had in mind ?

[blackbird7112]

Hey, I have been exploring this issue for a day. actions/checkout#124 (comment) This comment explains the reason why this issue arises. I think a push to a PR related branch is also considered as a trigger. So we need to do some study regarding when this workflow should be called.

The simplest way to fix the issue will be through a git checkout and then performing the push.

Also I had a suggestion. I think we need to setup a conditional statement to prevent the workflow from running in forks of the repository. This is because the fork will not have the BOT_TOKEN secret and the workflow may give an error.

I would love to work on this issue.

[blackbird7112]

I have a made a mistake in my previous comment. After testing the workflow on my fork, I have realised that the workflow is being triggered correctly and the PR is not causing the error. The detached head occurs due to the way git handles tags. When we make a commit over the tag, the commit is not attached to any branch. The tag still points to the previous commit. So we need to modify the tag accordingly to handle this change.

[epassaro]

Replaced by the pre-release workflow.

Closing.