Skip to content

Conversation

@erikmd
Copy link
Collaborator

@erikmd erikmd commented Oct 12, 2021

I tested this approach as realistically as possible in our dummy separate repo, but I guess the very test will be on the next release.
Anyway @yurug feel free to proofread this refactoring in case you'd see a typo somewhere…

* This wasn't automatic up to now because these jobs was in a separate
  workflow and the tagging or release events that occur in release.yml
  cannot trigger children workflows from a mere "secrets.GITHUB_TOKEN"

* href: https://docs.github.com/en/actions/learn-github-actions/events-that-trigger-workflows#triggering-new-workflows-using-a-personal-access-token
@erikmd
Copy link
Collaborator Author

erikmd commented Oct 13, 2021

Hi @yurug, I've proofread the workflow code once more and everything looks fine; so squash-merging this PR.

@erikmd erikmd merged commit 71c3590 into ocaml-sf:master Oct 13, 2021
@erikmd erikmd deleted the fix-docker-ci-cd branch October 13, 2021 15:01
@erikmd erikmd added the kind: infrastructure CI, build tools, development tools label Oct 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind: infrastructure CI, build tools, development tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant