-
Notifications
You must be signed in to change notification settings - Fork 2
Conversation
- name: GH deployment status (deactivate) | ||
id: gh_deployment | ||
uses: bobheadxi/deployments@v0.6.2 | ||
with: | ||
step: deactivate-env | ||
token: ${{ github.token }} | ||
env: ${{ env.APP_NAME }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should deactivate it?
Because we use the same env
for all the PRs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a good question, in the code that I was preparing for this I had the next line:
https://github.com/Qiskit/gh-actions/blob/clean-up-workflow/.github/workflows/code-engine-clean-up.yml#L44
But I'm not sure about which will be the final result for this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A little update about the change that we needed to merge after checking that this was not working as expected: #11
We can discuss the decision about the naming.
Co-authored-by: Abdón Rodríguez Davila <a@abdonrd.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
this PR creates a reusable workflow for cleaning up PR previews
Fixes: #9