-
Notifications
You must be signed in to change notification settings - Fork 82
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
fix(workflow): Fix globstar paths in labeler workflow and revise PR template #675
Conversation
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.
I think the maintenance overhead of this workflow has outweighed its usefulness so far.
It will be more valuable when we start auto-generating the release notes, but that's also something we still need to implement.
- '*Libraries/BlePhy/*' | ||
- '*Libraries/Cordio/*' | ||
- '*Tools/Bluetooth/*' | ||
- '**/Examples/**/BLE*/**/*' |
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.
Doesn't make sense that we would need to start this with a recursive glob. Assuming you've already tried Examples/**/...
and it didn't work I guess this is fine if it gets the workflow to behave.
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.
Usually, there's no need to, but I noticed the location of the runner when it checks out the HEAD can sometimes vary between inside or outside the directory repo
Description
Fixed the globstar paths for the labeler workflow. Hopefully, it should work now.
Also, revised the PR template.
Checklist Before Requesting Review