Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

add case statement for on_skipped #290

Merged
merged 1 commit into from
Jan 19, 2022
Merged

Conversation

raeyulca
Copy link
Contributor

Resolves #289

@sgotti
Copy link
Member

sgotti commented Jan 19, 2022

@raeyulca Great catch! Thanks for the PR. Merging.

We probably don't have some tests around this condition
@cirelli94 @alessandro-sorint Would you like to add some tests to check this case?

@alessandro-sorint
Copy link
Contributor

@sgotti I done the PR #302 with the test.
Can you accept my PR if it is ok? Thanks

tulliobotti64 pushed a commit to tulliobotti64/agola that referenced this pull request Oct 19, 2022
runconfig: add case statement for on_skipped
tulliobotti64 pushed a commit to tulliobotti64/agola that referenced this pull request Oct 19, 2022
runconfig: add case statement for on_skipped
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

on_skipped option doesn't work
3 participants