Skip to content

Conversation

supakeen
Copy link
Member

@supakeen supakeen commented Sep 26, 2025

Splits the tests into separate files along some arbitrary logical boundaries and runs them separately in GitHub CI so we have faster results and we could (theoretically) mark cross-arch as experimental.


Draft since I need to figure out CI and it will conflict a bunch of in-flight PRs. I'd like to merge this after those land so I don't cause extra work for others.

Will also require some changes to which actions are required :)

@supakeen supakeen force-pushed the split-tests branch 3 times, most recently from 09fb877 to 55cf012 Compare September 26, 2025 10:59
max-line-length=120

[MESSAGES CONTROL]
disable=R0801
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you use 'duplicate-code' instead? I had to look up what R0801 was :)

Copy link
Member Author

Choose a reason for hiding this comment

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

I could/should but it seems the .pylintrc is being ignored in CI anyhow so this might change/move :)

Splits the tests into separate files along some arbitrary logical
boundaries and runs them separately in GitHub CI so we have faster
results and we could (theoretically) mark cross-arch as experimental.

Signed-off-by: Simon de Vlieger <[email protected]>
Signed-off-by: Simon de Vlieger <[email protected]>
Signed-off-by: Simon de Vlieger <[email protected]>
Signed-off-by: Simon de Vlieger <[email protected]>
Signed-off-by: Simon de Vlieger <[email protected]>
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.

2 participants