-
Notifications
You must be signed in to change notification settings - Fork 24
fix: removing .git from inside git template urls #103
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #103 +/- ##
=======================================
Coverage 63.44% 63.44%
=======================================
Files 212 212
Lines 22308 22308
=======================================
+ Hits 14153 14154 +1
- Misses 7067 7068 +1
+ Partials 1088 1086 -2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
zimeg
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mwbrooks Such a clean change - great call saving this for a follow up with tests! 🙏 ✨
I left rambling thoughts on how this might snag an edge later, but we can explore this more if an issue appears perhaps 🔍
| }, | ||
| }, | ||
| "Returns the zip URL with .git suffix removed": { | ||
| templateURL: "https://github.com/slack-samples/deno-starter-template.git", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📣 This IMO is the case we ought consider as "correct" for the --template flag since it appears on the "HTTPS" tab of the Code dropdown from the sample page:
🔗 https://github.com/slack-samples/deno-starter-template
📺 https://github.com/slack-samples/deno-starter-template.git
FWIW we do not use the .git prefix in sample names but might recommend using the entire URL ending with .git if this is encountered later... Or recommend changing the sample name altogether, though I understand this can cause hassle 😉
|
Thanks for the review and thoughts @zimeg! I'm glad we now have a public issue tracker, so hopefully we'll catch any other edge cases and start to make recommendations for template names! 🔠 |
CHANGELOG
Summary
This pull request is a follow-up of #99 (comment) that fixes a bug when creating a project from a git repo template. If the repo name included
.gitinside of the name (and, optionally as the extension), then both.gitreferences would be removed from the URL.h/t @zimeg for noticing this bug! 🐛
Requirements