Skip to content

Refactoring: extract text transform library from libs/tags to libs/textutil - #3621

Merged
anton-107 merged 4 commits into
mainfrom
anton-107/textutil-transform
Sep 18, 2025
Merged

Refactoring: extract text transform library from libs/tags to libs/textutil#3621
anton-107 merged 4 commits into
mainfrom
anton-107/textutil-transform

Conversation

@anton-107

@anton-107anton-107 commented Sep 17, 2025

Copy link
Copy Markdown
Contributor

Why

Tests

Existing tests

@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Sep 17, 2025

Copy link
Copy Markdown
Collaborator

Run: 17823209402

Env✅​pass🙈​skip
✅​aws linux311528
✅​aws windows312527
✅​aws-ucws linux423426
✅​aws-ucws windows424425
✅​azure linux311527
✅​azure windows312526
✅​azure-ucws linux423425
✅​azure-ucws windows424424
✅​gcp linux310529
✅​gcp windows311528

@anton-107
anton-107 marked this pull request as ready for review September 17, 2025 15:08
Comment threadlibs/textutil/transform.go Outdated
Comment threadlibs/textutil/transform.go
@anton-107
anton-107 merged commit d2f6e17 into mainSep 18, 2025
12 of 13 checks passed
@anton-107
anton-107 deleted the anton-107/textutil-transform branch September 18, 2025 09:00
denik pushed a commit that referenced this pull request May 20, 2026
…/textutil` (#3621)
## Why
<!-- Why are these changes needed? Provide the context that the reviewer
might be missing.
For example, were there any decisions behind the change that are not
reflected in the code itself? -->
- Text transformation logic can be useful outside the domain of working
with tags
- I am currently planning to re-use it for populating a dns-friendly
short user name in a following PR:
#3623
## Tests
<!-- How have you tested the changes? -->
Existing tests
<!-- If your PR needs to be included in the release notes for next
release,
add a separate entry in NEXT_CHANGELOG.md as part of your PR. -->
Sign up for freeto 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.

4 participants

@anton-107@eng-dev-ecosystem-bot@pietern@andrewnester