Skip to content

Conversation

@joernu76
Copy link
Member

@joernu76 joernu76 commented Dec 3, 2025

Purpose of PR?:

Fixes two failing tests, one bug in test, one bug in documentation generation.

Does this PR introduce a breaking change?
no

Checklist:

  • Bug fix. Fixes #
  • New feature (Non-API breaking changes that adds functionality)
  • PR Title follows the convention of <type>: <subject>
  • Commit has unit tests

@joernu76 joernu76 force-pushed the fix_tests branch 2 times, most recently from 6b89e7f to d980a04 Compare December 3, 2025 08:06
Copy link
Member

@ReimarBauer ReimarBauer left a comment

Choose a reason for hiding this comment

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

This fix should go to stable

text = f.read().decode("utf-8")
expected_version = __version__
pattern = rf'value="is:closed milestone:{re.escape(expected_version)} "'
pattern = rf'value="is:closed milestone:{re.escape(expected_version)}"'
Copy link
Member

Choose a reason for hiding this comment

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

this is also wrong in stable. Because github got an update. This test finds updates on github.

please send this to stable

@joernu76 joernu76 changed the base branch from develop to stable December 3, 2025 10:31
the import needs to become added by the merge from stable to develop
Copy link
Member

@ReimarBauer ReimarBauer left a comment

Choose a reason for hiding this comment

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

thx

@ReimarBauer ReimarBauer merged commit 4368582 into stable Dec 3, 2025
10 of 12 checks passed
@ReimarBauer ReimarBauer mentioned this pull request Dec 5, 2025
annapurna-gupta pushed a commit to annapurna-gupta/MSS that referenced this pull request Jan 2, 2026
---------

Co-authored-by: ReimarBauer <[email protected]>
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