Skip to content

Deduplicate definition of ZEO cache trace records format#248

Merged
navytux merged 1 commit intomasterfrom
y/cache-fmt-dedup
Jan 28, 2026
Merged

Deduplicate definition of ZEO cache trace records format#248
navytux merged 1 commit intomasterfrom
y/cache-fmt-dedup

Conversation

@navytux
Copy link
Contributor

@navytux navytux commented Jan 28, 2026

Suggested by @bmwiedemann.

  • I signed and returned the Zope Contributor Agreement, and received and accepted an invitation to join a team in the zopefoundation GitHub organization.
  • I verified there aren't any other open pull requests for the same change.
  • I followed the guidelines in Developer guidelines.
  • I successfully ran code quality checks on my changes locally.
  • I successfully ran tests on my changes locally.
  • If needed, I added new tests for my changes.
  • If needed, I added documentation for my changes.
  • I included a change log entry in my commits.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 21440271260

Details

  • 4 of 4 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.01%) to 73.25%

Totals Coverage Status
Change from base Build 21439938517: 0.01%
Covered Lines: 7785
Relevant Lines: 10109

💛 - Coveralls

@navytux
Copy link
Contributor Author

navytux commented Jan 28, 2026

Sorry for several WIP-style pushes. The tests are now ok. And I do not think this patch needs a changelog entry.

@navytux
Copy link
Contributor Author

navytux commented Jan 28, 2026

Thanks, Jens.

@navytux navytux merged commit 611aff9 into master Jan 28, 2026
26 checks passed
@navytux navytux deleted the y/cache-fmt-dedup branch January 28, 2026 18:47
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