Skip to content

Fix markers being included as None/null#5788

Merged
oz123 merged 2 commits intomainfrom
issue-5786
Jul 11, 2023
Merged

Fix markers being included as None/null#5788
oz123 merged 2 commits intomainfrom
issue-5786

Conversation

@matteius
Copy link
Copy Markdown
Member

@matteius matteius commented Jul 10, 2023

The issue

Fixes #5786

The fix

use the empty string when the marker is None or null.

The checklist

  • Associated issue
  • A news fragment in the news/ directory to describe this fix with the extension .bugfix.rst, .feature.rst, .behavior.rst, .doc.rst. .vendor.rst. or .trivial.rst (this will appear in the release changelog). Use semantic line breaks and name the file after the issue number or the PR #.

@matteius matteius requested a review from oz123 July 10, 2023 19:07
@oz123 oz123 merged commit 57406bb into main Jul 11, 2023
@oz123 oz123 deleted the issue-5786 branch July 11, 2023 07:53
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.

Since 2023.7.1 requirements have wrong parameter

2 participants