Skip to content

Conversation

@mikeallenspach
Copy link
Contributor

No description provided.

@mikeallenspach mikeallenspach force-pushed the fix/unknown_pragma_warnings branch from 40cdc9a to 8becb6c Compare September 1, 2025 07:51
@mikeallenspach mikeallenspach requested a review from 4c3y September 1, 2025 08:03
@mikeallenspach
Copy link
Contributor Author

mikeallenspach commented Sep 1, 2025

@4c3y the CI seems to pass, but I have to restart every job manually and let them run one at a time.
It was always one of the *_timed unittests that failed. Locally everything works fine but maybe on the runners the timing is a bit different and that is why they fail.
Do you have an idea on how to fix this?

@4c3y
Copy link
Member

4c3y commented Sep 16, 2025

Fixed in #41

Copy link
Member

@4c3y 4c3y left a comment

Choose a reason for hiding this comment

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

See comment about formatting. Will review afterwards as the delta is very large.

@mikeallenspach
Copy link
Contributor Author

Formatting is fixed now

@mikeallenspach mikeallenspach requested a review from 4c3y September 22, 2025 09:08
@4c3y 4c3y merged commit 28873f2 into master Sep 23, 2025
47 checks passed
@4c3y 4c3y deleted the fix/unknown_pragma_warnings branch September 23, 2025 15:16
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