Skip to content
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

Use actual re.sub() regex-replace rather than str.replace(), since we… #81

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JeffSaxeVA
Copy link

… want to eliminate all varieties of whitespace from the Labels sent to Jira.

I noticed that the code to make a Label out of the alias name was evidently intending to remove all whitespace from the name, since spaces are not permitted in a Label on the Jira side. However, the str.replace() function recognizes only static strings, not character classes or other regex constructs; it was looking for literally backslash-s in the original text, so it would not even remove spaces. Since the "re" module is in the core library in both Python 2.x and 3.x, I propose importing that and using re.sub(). Thanks!

… want to eliminate all varieties of whitespace

from the Labels sent to Jira.
@ghost
Copy link

ghost commented Nov 23, 2022

Hi @JeffSaxeVA, you can track the progress about the issue from this ticket

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