Skip to content

fix(changelog-filters): fixes url resolution when prefix & path share letters #1239

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

Conversation

codejedi365
Copy link
Contributor

Purpose

Rationale

How did you test?

How to Verify


PR Completion Checklist

  • Reviewed & followed the Contributor Guidelines

  • Changes Implemented & Validation pipeline succeeds

  • Commits follow the Conventional Commits standard
    and are separated into the proper commit type and scope (recommended order: test, build, feat/fix, docs)

  • Appropriate Unit tests added/updated

  • Appropriate End-to-End tests added/updated

  • Appropriate Documentation added/updated and syntax validated for sphinx build (see Contributor Guidelines)

@codejedi365 codejedi365 force-pushed the fix/changelog-url-filter-generation branch 4 times, most recently from a402e7f to 31261ee Compare May 5, 2025 02:16
…are common letters

The change adjusts the url normalization computation to remove any
path prefix and only the path prefix even when letters are shared with
the rest of the url path.

Resolves: python-semantic-release#1204
@codejedi365 codejedi365 force-pushed the fix/changelog-url-filter-generation branch from 31261ee to ce45d5c Compare May 5, 2025 04:07
@codejedi365 codejedi365 merged commit f61f8a3 into python-semantic-release:master May 5, 2025
12 checks passed
@codejedi365 codejedi365 deleted the fix/changelog-url-filter-generation branch May 5, 2025 04:20
@codejedi365
Copy link
Contributor Author

🎉 This PR has been published as part of Version 9.21.1 🎉

You can find more information about this release on the GitHub Releases page.

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.

Unexpected & Invalid urls generated in changelog
1 participant