Skip to content

.github: Let's drop the --exit-zero tests that are being ignored#1069

Merged
amilcarlucas merged 2 commits intoArduPilot:masterfrom
cclauss:patch-15
Jul 3, 2025
Merged

.github: Let's drop the --exit-zero tests that are being ignored#1069
amilcarlucas merged 2 commits intoArduPilot:masterfrom
cclauss:patch-15

Conversation

@cclauss
Copy link
Contributor

@cclauss cclauss commented Jun 30, 2025

@cclauss cclauss changed the title .github: Let's drop the --exit-zero tests that are being ignored .github: Let's drop the --exit-zero tests that are being ignored Jun 30, 2025
@amilcarlucas amilcarlucas merged commit efeb2e0 into ArduPilot:master Jul 3, 2025
15 checks passed
Copy link
Contributor

@amilcarlucas amilcarlucas left a comment

Choose a reason for hiding this comment

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

But this completlly removes the second flake8 invocation

@cclauss cclauss deleted the patch-15 branch July 3, 2025 16:04
@khancyr
Copy link
Contributor

khancyr commented Jul 4, 2025

That is wrong, it removes the tests ...

@cclauss
Copy link
Contributor Author

cclauss commented Jul 4, 2025

The tests were always failing since they were implemented, but they were being ignored due to the --exit-zero option, which is why I included that URL in the commit message.

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.

3 participants