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

fix(tester): re-apply MacOS CI fix (commit 6fde964) #595

Merged
merged 1 commit into from
Jan 20, 2025

Conversation

niklasdewally
Copy link
Collaborator

@niklasdewally niklasdewally commented Jan 20, 2025

Reapply 6fde964 (fix(tester): make human rule trace writer blocking, 2025-01-08)

PR #569 accidentally reverted this change, causing ACCEPT=true failures
on my Mac and some CI failures.

This commit also adds the previous fix to the new JSON logging layer.

@niklasdewally niklasdewally force-pushed the nik/reapply-blocking-rule-trace-commit branch from 7e14ca6 to d077441 Compare January 20, 2025 10:50
@niklasdewally niklasdewally changed the title re-apply 6fde9643e (fix(tester): make human rule trace writer blocking, 2025-01-08) fix(tester): re-apply MacOS CI fix (commit 6fde964) Jan 20, 2025
@niklasdewally niklasdewally marked this pull request as draft January 20, 2025 10:51
Reapply 6fde964 (fix(tester): make human rule trace writer blocking,
2025-01-08)

PR #569 accidentally reverted this change, causing ACCEPT=true failures
on my Mac and some CI failures.

This commit also adds the previous fix to the new JSON logging layer.
@niklasdewally niklasdewally force-pushed the nik/reapply-blocking-rule-trace-commit branch from d077441 to 5f986f9 Compare January 20, 2025 10:56
@niklasdewally niklasdewally self-assigned this Jan 20, 2025
@niklasdewally niklasdewally marked this pull request as ready for review January 20, 2025 10:58
@ozgurakgun ozgurakgun merged commit 1fb1158 into main Jan 20, 2025
14 checks passed
@ozgurakgun ozgurakgun deleted the nik/reapply-blocking-rule-trace-commit branch January 20, 2025 11:37
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