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

Print all failed Requirements at once #1138

Open
tyb-dev opened this issue Mar 10, 2023 · 0 comments · May be fixed by #1159
Open

Print all failed Requirements at once #1138

tyb-dev opened this issue Mar 10, 2023 · 0 comments · May be fixed by #1159
Assignees

Comments

@tyb-dev
Copy link
Contributor

tyb-dev commented Mar 10, 2023

Description

The current RequirementService.ValidateRequirements implementation fails on the first requirement that fails. This means the user needs to retry a couple of times until fulfilling all the requirements. Since all requirements are evaluated in for foreach loops, Nuke could as well collect all failures in a collection and report all of them at once.
This would be possible at least in the case where NukeBuild.Host is not a Terminal and there is no interactive injection of values.

Usage Example

Usage is the same, but printed output is more useful.

Alternative

The user has to do a try & error.

Could you help with a pull-request?

Yes

@matkoch matkoch removed the triage label Apr 2, 2023
@matkoch matkoch linked a pull request May 7, 2023 that will close this issue
3 tasks
@matkoch matkoch added this to the v7.1.0 milestone May 7, 2023
@matkoch matkoch removed this from the v8.0.0 milestone Nov 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants