-
-
Notifications
You must be signed in to change notification settings - Fork 85
Open
Labels
status: accepting prsPlease, send a pull request to resolve this!Please, send a pull request to resolve this!type: featureNew enhancement or requestNew enhancement or request
Milestone
Description
Bug Report Checklist
- I have pulled the latest
mainbranch of the repository. - I have searched for related issues and found none that matched my issue.
Overview
The total time to create a repository using create-typescript-app's Common preset right now is about 35 seconds. That's rather slow. I'd like to get it down much shorter.
As part of bingo-js/bingo#72, I added in rudimentary timing groups for applied scripts and requests. Here's raw data from an example run with create-typescript-app's Common preset:
┌────────────┬─────────────────────────────────────────────────┬───────────┐
│ group │ id │ time (ms) │
├────────────┼─────────────────────────────────────────────────┼───────────┤
│ 'scripts' │ 'pnpm install' │ 6239 │
│ 'scripts' │ 'pnpm lint --fix' │ 2305 │
│ 'scripts' │ 'npx -y all-contributors-cli generate' │ 13424 │
│ 'scripts' │ 'npx -y all-contribut ... ojectManagement,tool' │ 12355 │
│ 'scripts' │ 'pnpm format --write' │ 792 │
│ 'requests' │ 'branch-protection' │ 149 │
│ 'requests' │ 'repository-labels' │ 16452 │
│ 'requests' │ 'repository-settings' │ 1335 │
└────────────┴─────────────────────────────────────────────────┴───────────┘
Additional Info
My initial takeaways are that:
npx -y all-contributors-clicommands are slow: I'd like to look into doing their work manually- This isn't the first time I've been tempted to: Node API? all-contributors/cli#362
repository-labels, runningsetGitHubRepositoryLabelsfromset-github-repository-labels, is much slower than it should be too. I'll have to look at its performance again.- Can its multiple API calls be joined into one?
- Failing that, can it be parallelized more?
pnpm installtaking >6 seconds also isn't ideal. Can I speed that up?- Does
pnpm store addhave any impact? I'm guessing no, that that would only serve to bring an even-higher install time down to the >6 seconds I'm seeing...
- Does
💖
Metadata
Metadata
Assignees
Labels
status: accepting prsPlease, send a pull request to resolve this!Please, send a pull request to resolve this!type: featureNew enhancement or requestNew enhancement or request