-
Notifications
You must be signed in to change notification settings - Fork 30
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
Convert the Action into TypeScript #101
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.
Looking real close, but the nit about cd
. I'll send a fixup.
a9e2170
to
1fd4446
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.
Doesn't seem to work as-is for me; no PR gets created.
https://github.com/cole-h/update-flake-lock-test/actions/runs/9020862190/job/24786935346 was run from the HEAD (at the time) of this branch -- notice how no PR was created, as evidenced by the lack of a number in the "echo" step:
https://github.com/cole-h/update-flake-lock-test/actions/runs/9020902721/job/24787061455 was run off of main
, a PR was created, and the echo step has a PR number:
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.
LGTM, but we should b e prepared to receive issues over the next while about this breaking things that were not tested very deeply.
Agreed. |
This PR converts the Action to TypeScript using our
detsys-ts
framework.