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

Try enforcing required #2059

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Try enforcing required #2059

wants to merge 1 commit into from

Conversation

jsoref
Copy link
Contributor

@jsoref jsoref commented Aug 17, 2022

I'm hoping this could address #1070

But I can't figure out how to write a test for it based on the current testing bits.

It'd be conceptually easy to have a proper action file, roughly:
https://github.com/actions/runner/blob/be9632302ceef50bfb36ea998cea9c94c75e5d4d/src/Test/TestData/nodeaction.yml
minus:

default: 'Hello'
deprecationMessage: 'This property has been deprecated'

And then have a workflow which referenced that action, but didn't provide a greeting.

But, I can't figure out how to go from where the C# unit tests are to a point that would either do that, or do the equivalent of that...

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
@jsoref jsoref requested a review from a team as a code owner August 17, 2022 06:23
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.

None yet

1 participant