Skip to content

Move leeway vet to validate #12999

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

Merged
merged 1 commit into from
Sep 15, 2022
Merged

Move leeway vet to validate #12999

merged 1 commit into from
Sep 15, 2022

Conversation

mads-hartmann
Copy link
Contributor

@mads-hartmann mads-hartmann commented Sep 15, 2022

Description

This follows up on #12998 and adds leeway vet to the Validating changes phase too to catch such issues early on.

It also means we save 7 seconds as this was otherwise running sequentially as part of the build

Screenshot 2022-09-15 at 15 06 02

Related Issue(s)

No issue, just feeling like doing some minor cleanups.

How to test

werft job run github

Release Notes

NONE

Documentation

N/A

Werft options:

  • /werft with-preview

@mads-hartmann mads-hartmann force-pushed the mads/move-leeway-vet-to-validate branch from 41d6ad6 to 7dc93e2 Compare September 15, 2022 13:00
@mads-hartmann mads-hartmann marked this pull request as ready for review September 15, 2022 13:07
Copy link
Contributor

@ArthurSens ArthurSens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@roboquat roboquat merged commit 3125118 into main Sep 15, 2022
@roboquat roboquat deleted the mads/move-leeway-vet-to-validate branch September 15, 2022 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants