-
Notifications
You must be signed in to change notification settings - Fork 5
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
Use github-script to generate versioned blobs and commits #1301
Conversation
612ada1
to
5049b23
Compare
5049b23
to
57caecf
Compare
57caecf
to
afe761d
Compare
afe761d
to
7fa1e3a
Compare
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.
Just left some minor comments, nothing that should hold back the merge.
Don't really know the GitHub API though. Took a quick look at the documentation and what you have seems correct. I assume the PR would not merge otherwise anyway
7c75cf0
to
355e372
Compare
Testing PR open, sync, and merge here: balena-io-experimental#12 |
Post-merge workflow test run: https://github.com/balena-io-experimental/flowzone/actions/runs/12562179536 |
Change-type: patch Signed-off-by: Kyle Harding <[email protected]>
355e372
to
27aaed5
Compare
Merge workflow looks good, everything is versioned and tagged correctly. I've squashed the commits and it's ready to go! |
LGTM |
Resolves 10 code-scanning warnings.