Skip to content

chore: Remove Pyzor + Razor packages#4548

Merged
georglauterbach merged 4 commits intomasterfrom
gl/remove-pyzor-razor
Aug 17, 2025
Merged

chore: Remove Pyzor + Razor packages#4548
georglauterbach merged 4 commits intomasterfrom
gl/remove-pyzor-razor

Conversation

@georglauterbach
Copy link
Copy Markdown
Member

@georglauterbach georglauterbach commented Aug 10, 2025

Description

This PR removes Pyzor and Razor, which are not actually in use according to #4351. This change also simplifies the build process again.

Fixes #4351

UPDATE: Pyzor was found to be still working, however it would ignore any digest with high report/whitelist counts and the project itself seems to lack proper maintenance.

Type of change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (README.md or the documentation under docs/)
  • If necessary, I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have added information about changes made in this PR to CHANGELOG.md

@georglauterbach georglauterbach added this to the v16.0.0 milestone Aug 10, 2025
@georglauterbach georglauterbach self-assigned this Aug 10, 2025
@georglauterbach georglauterbach added area/features kind/update Update an existing feature, configuration file or the documentation service/security/spamassassin labels Aug 10, 2025
@georglauterbach georglauterbach marked this pull request as draft August 10, 2025 17:42
@georglauterbach georglauterbach marked this pull request as ready for review August 12, 2025 21:55
Signed-off-by: Georg Lauterbach <[email protected]>
Signed-off-by: Georg Lauterbach <[email protected]>
@georglauterbach
Copy link
Copy Markdown
Member Author

This PR is now ready for review :)

@georglauterbach georglauterbach moved this to Implementation Phase in DMS Features & Tasks Aug 13, 2025
@georglauterbach georglauterbach moved this from Implementation Phase to In Review in DMS Features & Tasks Aug 13, 2025
casperklein
casperklein previously approved these changes Aug 16, 2025
Copy link
Copy Markdown
Member

@polarathene polarathene left a comment

Choose a reason for hiding this comment

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

I don't have time to complete the investigation in the associated issue but this change seems like it'd be fine 👍

Comment thread CHANGELOG.md Outdated
@polarathene polarathene changed the title chore: remove Pyzor & Razor chore: Remove Pyzor + Razor packages Aug 16, 2025
@github-actions
Copy link
Copy Markdown
Contributor

Documentation preview for this PR is ready! 🎉

Built with commit: b244d6b

@georglauterbach georglauterbach merged commit da61eb2 into master Aug 17, 2025
14 of 15 checks passed
@georglauterbach georglauterbach deleted the gl/remove-pyzor-razor branch August 17, 2025 09:00
@github-project-automation github-project-automation Bot moved this from In Review to Done in DMS Features & Tasks Aug 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/features kind/update Update an existing feature, configuration file or the documentation service/security/spamassassin

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Assess relevance of keeping Pyzor + Razor

3 participants