-
Notifications
You must be signed in to change notification settings - Fork 5
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
Slugify branch names in patchwork report #39
Conversation
@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Thanks. Do you mind putting this in a function and re-using the function in those 2 places? There is probably more char that we need to properly replace that we may find out later. |
@liu-song-6 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@chantra has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@liu-song-6 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@liu-song-6 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
patchwork only accepts ASCII letters, numbers, underscores or hyphens in slugs. Let's replace "." in the branch name, for example md-6.10, with "_", i.e. md-6_10. Signed-off-by: Song Liu <[email protected]>
@liu-song-6 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@liu-song-6 merged this pull request in 454b980. |
patchwork only accepts ASCII letters, numbers, underscores or hyphens in slugs. Let's replace "." in the branch name, for example md-6.10, with "_", i.e. md-6_10.