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

Refactor in preparation for dependency-graph support #734

Merged
merged 3 commits into from
Jun 6, 2023
Merged

Conversation

bigdaz
Copy link
Member

@bigdaz bigdaz commented Jun 5, 2023

No description provided.

bigdaz added 3 commits June 3, 2023 14:09
Introducing new actions for the GitHub dependency graph will involve reuse of much of
the action infrastructure. This commit reorganises things a little to facilitate reuse.
Moved reading of all input parameters into a common source: `input-params.ts`.
This centralized all input parameter reads, and allowed an improved implementation
of reading boolean parameters. In particular, the implementation now provides a default
value for a boolean input parameter that isn't declared for an action.
@bigdaz bigdaz force-pushed the dd/input-params branch from bb04a39 to 4388b1f Compare June 6, 2023 16:52
@bigdaz bigdaz merged commit 1b2daf5 into main Jun 6, 2023
@bigdaz bigdaz deleted the dd/input-params branch June 6, 2023 16:52
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