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

[Transformer] NormalizeTags transformer #212

Closed
adrszad opened this issue Sep 24, 2021 · 3 comments
Closed

[Transformer] NormalizeTags transformer #212

adrszad opened this issue Sep 24, 2021 · 3 comments

Comments

@adrszad
Copy link
Contributor

adrszad commented Sep 24, 2021

Transformer for cleaning the tags, by converting them to same case and removing duplicates.
What should be the default case and which other cases to support? Lowercase?

This library seems to be handy if we want to support more cases:
https://pypi.org/project/stringcase/

Optional feature:

With extra option "common_to_force_tags" (temporary name) that would move tags that are repeated in every test case to Force Tags.

Originally posted by @bhirsz in #205 (comment)

@bhirsz
Copy link
Member

bhirsz commented Sep 24, 2021

Interesting library - could come in handy with my other normalizers. Although for now we can stick to cases you implemented in PR - with lowercase as default.

Another idea is providing parameter for ignoring (not normalizing) some of the test cases. For example [A-Z]+-[0-9]+ for skipping 'JIRA-123' tags. But it's idea for the future.

@bhirsz
Copy link
Member

bhirsz commented Sep 28, 2021

@adrszad I have merge the code and I will create release tommorow. Do you think we can close this issue? (we can create another one if we want to extend this transformer)

@adrszad
Copy link
Contributor Author

adrszad commented Sep 29, 2021

@adrszad I have merge the code and I will create release tommorow. Do you think we can close this issue? (we can create another one if we want to extend this transformer)

yes, let's do it, I'll try to handle both excluding tags by regex and moving common to force tags

@bhirsz bhirsz closed this as completed Sep 30, 2021
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

No branches or pull requests

2 participants