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

Update load-used-actions.ps1 #69

Merged
merged 30 commits into from
Sep 11, 2023
Merged

Update load-used-actions.ps1 #69

merged 30 commits into from
Sep 11, 2023

Conversation

rajbos
Copy link
Collaborator

@rajbos rajbos commented Sep 7, 2023

What are the incoming changes?

🤖 Generated by Copilot at 9f1f1cb

This pull request enhances the load-used-actions.ps1 script to make it more robust and easier to troubleshoot. It adds validation, error handling, and debugging features to the script.

🤖 Generated by Copilot at 9f1f1cb

We face the doom of broken workflows
We validate the YAML with our skills
We fix the bug that made us lose control
We debug the script and unleash our wills

Why are they needed

How has this been achieved?

🤖 Generated by Copilot at 9f1f1cb

  • Add validation and error handling to workflow file content before parsing it as YAML (link)
  • Return actions array instead of nothing in catch block to ensure consistency in return type (link)
  • Comment out continue statement in inner loop catch block to fix bug that caused script to break (link)
  • Add debug statement to print "here!" to console for troubleshooting (link)

@github-actions
Copy link
Contributor

github-actions bot commented Sep 7, 2023

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Manifest Files

@rajbos rajbos marked this pull request as ready for review September 8, 2023 11:00
@rajbos rajbos enabled auto-merge (squash) September 11, 2023 15:44
@rajbos rajbos merged commit 30d0ad2 into main Sep 11, 2023
9 checks passed
@rajbos rajbos deleted the updated-after-test branch September 11, 2023 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant