Skip to content

Conversation

@pohly
Copy link
Contributor

@pohly pohly commented Oct 21, 2025

This mirrors the more recent GitHub setup from the logr repo which was more actively maintained and updated than the copy in zapr. The setup here suffered from bitrot and no longer worked.

pohly added 3 commits October 21, 2025 12:50
This mirrors the more recent GitHub setup from the logr repo which was more
actively maintained and updated than the copy in zapr. The setup here suffered
from bitrot and no longer worked.
It's no longer supported by the Go team and lacks atomic.Bool, which is
required by one of the zapr dependencies.
@pohly pohly changed the title GitHub: sync with logr repo GitHub: fix actions Oct 21, 2025
@pohly pohly force-pushed the fix-github-actions branch from 76553e7 to c1d9757 Compare October 21, 2025 11:24
pohly and others added 3 commits October 21, 2025 13:27
slogtest.Run was added long ago enough that we can rely on it by
dropping support for older Go releases.
The API was moved to the lop-level logr repo.
@pohly pohly force-pushed the fix-github-actions branch from c1d9757 to a7113e4 Compare October 21, 2025 11:27
@pohly
Copy link
Contributor Author

pohly commented Oct 21, 2025

@nilebox: does this look reasonable?

@pohly
Copy link
Contributor Author

pohly commented Oct 22, 2025

/cc @thockin

The slogtest.Run change is something that we might want to do also in logr/logr.

@thockin
Copy link
Contributor

thockin commented Oct 22, 2025

/approve

@pohly pohly merged commit 6ecfdb2 into go-logr:master Oct 23, 2025
11 checks passed
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