Skip to content
This repository has been archived by the owner on Oct 17, 2023. It is now read-only.

fix: Ensure documentation workflows behave as expected #169

Merged
merged 1 commit into from
Jun 1, 2023

Conversation

phated
Copy link
Contributor

@phated phated commented May 25, 2023

This updates the documentation workflows so they behave correctly. I found that the "delete" workflow was actually attempting to close every migrated issue that was ever created. I also cleaned up the "new" workflow so it will re-open an issue if the "doc needed" label is toggled on the upstream PR.

@phated phated requested a review from signorecello May 25, 2023 21:28
@netlify
Copy link

netlify bot commented May 25, 2023

Deploy Preview for noir-docs ready!

Name Link
🔨 Latest commit f5cbc9c
🔍 Latest deploy log https://app.netlify.com/sites/noir-docs/deploys/646fd31a72e09f000828d7b5
😎 Deploy Preview https://deploy-preview-169--noir-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@critesjosh
Copy link
Collaborator

@phated was this issue closed as part of your testing? #146

I don't think it's complete yet.

@phated
Copy link
Contributor Author

phated commented May 26, 2023

@critesjosh I believe that issue was closed by the broken workflow, which is what showed me these workflows had a problem. I'll re-open it.

Copy link
Collaborator

@signorecello signorecello left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@signorecello signorecello merged commit e62ce2e into master Jun 1, 2023
@signorecello signorecello deleted the phated/fix-issue-workflows branch June 1, 2023 20:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants