Skip to content

[INFRA] Improve the bpmn-visualization update workflow#322

Merged
tbouffard merged 1 commit into
masterfrom
infra/improve_update_bpmn-visu_wf
Apr 25, 2022
Merged

[INFRA] Improve the bpmn-visualization update workflow#322
tbouffard merged 1 commit into
masterfrom
infra/improve_update_bpmn-visu_wf

Conversation

@tbouffard
Copy link
Copy Markdown
Member

Apply changes recently tested in the playground repository.

The major change is the usage of GH_RELEASE_TOKEN instead of GITHUB_TOKEN with the dawidd6/action-download-artifact.
According to the action documentation, this lets the action access to artifacts stored in another repository. As we know
this is working with more restricted GITHUB_TOKEN permissions in the playground repository, let's apply the change now
instead of waiting for an issue when the permissions restriction will be in place in the examples repository.

Resources

Apply changes recently tested in the playground repository.

The major change is the usage of `GH_RELEASE_TOKEN` instead of `GITHUB_TOKEN` with the dawidd6/action-download-artifact.
According to the action documentation, this lets the action access to artifacts stored in another repository. As we know
this is working with more restricted `GITHUB_TOKEN` permissions in the playground repository, let's apply the change now
instead of waiting for an issue when the permissions restriction will be in place in the `examples` repository.
@tbouffard tbouffard added the chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...) label Apr 25, 2022
@tbouffard tbouffard requested a review from csouchet April 25, 2022 07:27
@tbouffard tbouffard merged commit 188baf3 into master Apr 25, 2022
@tbouffard tbouffard deleted the infra/improve_update_bpmn-visu_wf branch April 25, 2022 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Build, CI/CD or repository tasks (issues/PR maintenance, environments, ...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants