Skip to content

Conversation

@NevilleS
Copy link
Contributor

@NevilleS NevilleS commented Feb 9, 2026

Ticket ENG-2516

Description Of Changes

Add a shared clean_version() function to fides.common.utils that removes noisy suffixes from version strings for cleaner user-facing display. This enables fidesplus to import and reuse this utility rather than duplicating the code.

Code Changes

  • Added clean_version() function to src/fides/common/utils.py that strips:
    • The dirty suffix (.dirty or -dirty) added when there are uncommitted changes
    • The +0.gXXXXXX suffix when exactly on a git tag (zero commits past)
  • Updated src/fides/__init__.py to import clean_version from common.utils instead of defining it locally
  • Added comprehensive unit tests in tests/lib/test_version.py

Steps to Confirm

  1. Verify clean_version can be imported from fides.common.utils
  2. Run nox -s "pytest(lib)" -- tests/lib/test_version.py to verify tests pass
  3. Confirm version display remains clean in health endpoints

Pre-Merge Checklist

  • Issue requirements met
  • All CI pipelines succeeded
  • CHANGELOG.md updated
    • Add a db-migration This indicates that a change includes a database migration label to the entry if your change includes a DB migration
    • Add a high-risk This issue suggests changes that have a high-probability of breaking existing code label to the entry if your change includes a high-risk change (i.e. potential for performance impact or unexpected regression) that should be flagged
    • Updates unreleased work already in Changelog, no new entry necessary
  • UX feedback:
    • No UX review needed
  • Followup issues:
    • Followup issues created
    • No followup issues
  • Database migrations:
    • No migrations
  • Documentation:
    • No documentation updates required

Made with Cursor

Add a shared clean_version() function to fides.common.utils that removes
noisy suffixes from version strings for cleaner display. This strips:
- dirty suffix (.dirty or -dirty) from uncommitted changes
- +0.gXXXXXX suffix when exactly on a tag

Update fides/__init__.py to import and use the shared utility.
Add comprehensive unit tests for the clean_version function.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel
Copy link
Contributor

vercel bot commented Feb 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
fides-plus-nightly Ignored Ignored Feb 9, 2026 6:25pm
fides-privacy-center Ignored Ignored Feb 9, 2026 6:25pm

Request Review

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.

1 participant