-
Notifications
You must be signed in to change notification settings - Fork 14
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
Refactor create-app dotfile generation #31
Conversation
Generate changelog in
|
osdkPackage: string; | ||
osdkRegistryUrl: string; | ||
}): string { | ||
const withoutProtocol = osdkRegistryUrl.replace(/^https:\/\//, ""); |
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.
I guess this is safe given promptOsdkRegistryUrl
already tests it against a regex. If we plan to use this somewhere that isn't part of this code path, should we consider passing the protocol but have it with a default value?
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.
I think the only other protocol that would be relevant would be http
but should be safe to only allow https
for this url
* origin/main: Autorelease Increase CLI max width to 150 columns (#47) Debug log site zip stats (#49) Clean up yargs and consola imports (#51) Autorelease Add generated changelog entries Breaking: force dollar sign in osdk verbs for where Display errors more consistently in @osdk/cli (#41) Fix tagPrefix Add generated changelog entries Create foundry.config.json in create-app [3/] Improvement: Use loadedToken for fetching data (#17) Move argv override logging to middleware (#42) Make sure to run all tests (#39) Update create-app to use FOUNDRY_TOKEN in templates (#40) Add generated changelog entries Fix intellisense for aggregations and fill in more number types [2/] Improvement: Introduce new command structure (#16) Autorelease Refactor create-app prompts with tests (#32) Refactor create-app dotfile generation (#31) add changelog Add generated changelog entries Add generated changelog entries put mkdir in right place remove generated files add ignore for legacy client remove deleted file added add back mkdir move mkdir fix fixes keep trying remove module shared testing infra for legacy
TemplateContext
type to make it clearer what variables are available during templatingThe .npmrc / .env files are shared across all templates but could technically be in the individual template folders. For now I've kept the structure the same as before