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

Convert the Action into TypeScript #101

Merged
merged 19 commits into from
May 9, 2024
Merged

Convert the Action into TypeScript #101

merged 19 commits into from
May 9, 2024

Conversation

lucperkins
Copy link
Member

@lucperkins lucperkins commented Apr 26, 2024

This PR converts the Action to TypeScript using our detsys-ts framework.

@lucperkins lucperkins marked this pull request as ready for review April 26, 2024 17:22
grahamc
grahamc previously requested changes May 6, 2024
Copy link
Member

@grahamc grahamc left a 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.

src/index.ts Outdated Show resolved Hide resolved
src/index.ts Outdated Show resolved Hide resolved
action.yml Outdated Show resolved Hide resolved
@grahamc grahamc dismissed their stale review May 7, 2024 14:26

stale

action.yml Outdated Show resolved Hide resolved
@grahamc grahamc force-pushed the detsys-ts branch 2 times, most recently from a9e2170 to 1fd4446 Compare May 8, 2024 03:01
@grahamc grahamc requested a review from cole-h May 8, 2024 03:05
Copy link
Member

@cole-h cole-h left a 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:

image

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:

image

action.yml Outdated Show resolved Hide resolved
src/index.ts Outdated Show resolved Hide resolved
src/index.ts Outdated Show resolved Hide resolved
Copy link
Member

@cole-h cole-h left a 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.

@grahamc
Copy link
Member

grahamc commented May 9, 2024

Agreed.

@grahamc grahamc merged commit bcabaab into main May 9, 2024
2 checks passed
@grahamc grahamc deleted the detsys-ts branch May 9, 2024 19:50
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.

3 participants