Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

verify {input} is used by target_env or target_options in coverage task #1097

Merged
merged 7 commits into from
Jul 21, 2021

Conversation

bmc-msft
Copy link
Contributor

No description provided.

Co-authored-by: Cheick Keita <kcheick@gmail.com>
@bmc-msft bmc-msft requested a review from chkeita July 21, 2021 17:36
@ranweiler
Copy link
Member

ranweiler commented Jul 21, 2021

See also: #925. I think we also want to validate this at the service layer, though, when creating the task (so let's keep that issue open).

@bmc-msft
Copy link
Contributor Author

@ranweiler, this was more specifically to check for the local fuzzing mode. I agree the service should handle this on task creation which is why I left #925 open.

@bmc-msft bmc-msft merged commit 75d7120 into microsoft:main Jul 21, 2021
@bmc-msft bmc-msft deleted the verify-input-is-set-in-args branch July 21, 2021 20:41
@ghost ghost locked as resolved and limited conversation to collaborators Aug 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants