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

Adjust release merge workflow #4926

Merged
merged 1 commit into from
Dec 1, 2021

Conversation

bdukes
Copy link
Contributor

@bdukes bdukes commented Dec 1, 2021

The workflow triggered by the merge/rebase of #4925 failed, presumably because it was accessing the merge_commit_sha but the PR was closed via rebase. I've updated the workflow to just use sha, which still points to the rebased commit, and tested on my repo.

@bdukes bdukes added this to the 9.10.3 milestone Dec 1, 2021
bdukes added a commit to bdukes/Dnn.Platform that referenced this pull request Dec 1, 2021
@valadas
Copy link
Contributor

valadas commented Dec 1, 2021

This is a release management task, as per our policy, we are self-approving it.

@valadas valadas merged commit bde54cd into dnnsoftware:develop Dec 1, 2021
@bdukes
Copy link
Contributor Author

bdukes commented Dec 2, 2021

@valadas @mitchelsellers it looks like there's still a permissions issue (I guess on my test repo opening a PR for my own repo didn't trigger this scenario). I think we may need to add a secret with an access token that has permission to create branches, and use that instead of GITHUB_TOKEN.

@valadas
Copy link
Contributor

valadas commented Dec 2, 2021

I don't know, I have done it with GITHUB_TOKEN for the updating of versions one...

@bdukes
Copy link
Contributor Author

bdukes commented Dec 2, 2021

I saw someone saying that GitHub won't allow editing workflow files with GITHUB_TOKEN, so maybe it's just that develop currently has some changes to the workflow files…

@bdukes
Copy link
Contributor Author

bdukes commented Dec 2, 2021

I guess it just successfully created #4934, so maybe it has to do with who created the PR it's reacting to?

@valadas
Copy link
Contributor

valadas commented Dec 2, 2021

@valadas valadas modified the milestones: 9.10.3, 9.11.0 Sep 28, 2022
@bdukes bdukes deleted the fix-release-merge-workflow branch January 19, 2023 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants