-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Fix double dots in template file name extensions #33520
Conversation
Fixes Azure#33515 and may be solution or work around to dotnet/sdk#29962
This may fix or at least work around the problem. In testing with |
Note: I also formatted the document since editors like VSCode are going to at least reformat code one modifies. In general, it's best to keep code formatted so frequent re-formatting doesn't happen. If you look at the PR and hide whitespace changes, you'll find it's very small. |
Hello @heaths I tried this solution in test env, it works. After the PR is merged, I will try it in prod env. Thanks. |
@chunyu3 can you test this again? I generated a new version and checked a few files' contents, but a more thorough test like you did last night would be great. |
Fixes #33515 and may be solution or work around to dotnet/templating#5955