Fix the script bug in importing-a-subversion-repository.md#38552
Fix the script bug in importing-a-subversion-repository.md#38552Sharra-writes merged 2 commits intogithub:mainfrom
Conversation
Fix the bug in the author extraction script that causes author's starting from 't' losing it, e.g. SVN log: ``` r5572 | toe | 2023-04-10 09:25:13 -0400 (Mon, 10 Apr 2023) `` Result mapping: ``` om = tom ``` The corrected script produces the expected mapping: ``` tom = tom ```
|
Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines. |
How to review these changes 👓Thank you for your contribution. To review these changes, choose one of the following options: A Hubber will need to deploy your changes internally to review. Table of review linksNote: Please update the URL for your staging server or codespace. The table shows the files in the
Key: fpt: Free, Pro, Team; ghec: GitHub Enterprise Cloud; ghes: GitHub Enterprise Server 🤖 This comment is automatically generated. |
|
Thanks for opening a PR! I may have to go digging for an SME on this one. It's not a subject we encounter very often! I'll let you know what I find. |
|
I am migrating a bunch of our Subversion repositories and noticed that one of the authors with a user name starting from 't' losing it. I am executing commands from a terminal on Mac, but if I put this command |
jeviet
left a comment
There was a problem hiding this comment.
Validating that the updated script produces the expected results
|
@nih-dk Sorry that took some time, and thanks for your patience. I'll get this added to the merge queue once checks are passing. |
|
Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. If you're looking for your next contribution, check out our help wanted issues ⚡ |
Fix the bug in the author extraction script that causes authors starting from 't' losing it, e.g.
SVN log:
Result mapping:
The corrected script produces the expected mapping:
Why:
Closes: #38551
What's being changed (if available, include any code snippets, screenshots, or gifs):
Check off the following: