Skip to content
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

chore(core): replace shell calls by simple git #10363

Closed
wants to merge 1 commit into from
Closed

Conversation

OzakIOne
Copy link
Collaborator

@OzakIOne OzakIOne commented Aug 1, 2024

Motivation

Replace shell calls for git with simple-git

Fix #10348 ?

Test Plan

Unit tests + preview

Test links

https://deploy-preview-10363--docusaurus-2.netlify.app/

@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Aug 1, 2024
Copy link

New dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@argos-ci/playwright@1.9.3 Transitive: environment, filesystem +5 172 kB neoziro
npm/@netlify/functions@1.6.0 None +1 20.6 kB netlify-bot
npm/@playwright/test@1.45.3 None 0 25.4 kB yurys

View full report↗︎

Copy link

netlify bot commented Aug 1, 2024

[V2]

Name Link
🔨 Latest commit 3c7d65b
🔍 Latest deploy log https://app.netlify.com/sites/docusaurus-2/deploys/66ab93269ceb770008b3f4af
😎 Deploy Preview https://deploy-preview-10363--docusaurus-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

github-actions bot commented Aug 1, 2024

⚡️ Lighthouse report for the deploy preview of this PR

URL Performance Accessibility Best Practices SEO Report
/ 🟠 70 🟢 98 🟢 96 🟢 100 Report
/docs/installation 🟠 52 🟢 97 🟢 100 🟢 100 Report
/docs/category/getting-started 🟠 75 🟢 100 🟢 100 🟠 86 Report
/blog 🟠 68 🟢 96 🟢 100 🟠 86 Report
/blog/preparing-your-site-for-docusaurus-v3 🟠 54 🟢 92 🟢 100 🟢 100 Report
/blog/tags/release 🟠 69 🟢 96 🟢 100 🟠 86 Report
/blog/tags 🟠 75 🟢 100 🟢 100 🟠 86 Report

Copy link

github-actions bot commented Aug 1, 2024

Size Change: -1.04 kB (-0.01%)

Total Size: 10.9 MB

ℹ️ View Unchanged
Filename Size Change
website/.docusaurus/codeTranslations.json 2 B 0 B
website/.docusaurus/docusaurus.config.mjs 27.4 kB 0 B
website/.docusaurus/globalData.json 30 kB 0 B
website/.docusaurus/i18n.json 930 B 0 B
website/.docusaurus/registry.js 147 kB 0 B
website/.docusaurus/routes.js 71.6 kB 0 B
website/.docusaurus/routesChunkNames.json 75 kB 0 B
website/.docusaurus/site-metadata.json 2.17 kB 0 B
website/build/assets/css/styles.********.css 112 kB 0 B
website/build/assets/js/main.********.js 668 kB 0 B
website/build/blog.html 62.2 kB 0 B
website/build/blog/2017/12/14/introducing-docusaurus.html 74.8 kB -14 B (-0.02%)
website/build/blog/2018/04/30/How-I-Converted-Profilo-To-Docusaurus.html 47.2 kB -1 B (0%)
website/build/blog/2018/09/11/Towards-Docusaurus-2.html 51.5 kB -1 B (0%)
website/build/blog/2018/12/14/Happy-First-Birthday-Slash.html 33.2 kB -14 B (-0.04%)
website/build/blog/2019/12/30/docusaurus-2019-recap.html 39.5 kB -14 B (-0.04%)
website/build/blog/2020/01/07/tribute-to-endi.html 34 kB -14 B (-0.04%)
website/build/blog/2021/01/19/docusaurus-2020-recap.html 48.9 kB -14 B (-0.03%)
website/build/blog/2021/03/09/releasing-docusaurus-i18n.html 45.1 kB -14 B (-0.03%)
website/build/blog/2021/05/12/announcing-docusaurus-two-beta.html 52 kB -14 B (-0.03%)
website/build/blog/2021/11/21/algolia-docsearch-migration.html 52.9 kB -1 B (0%)
website/build/blog/2022/01/24/docusaurus-2021-recap.html 44.4 kB -14 B (-0.03%)
website/build/blog/2022/08/01/announcing-docusaurus-2.0.html 142 kB -14 B (-0.01%)
website/build/blog/2022/09/01/docusaurus-2.1.html 50.8 kB -14 B (-0.03%)
website/build/blog/archive.html 24.6 kB 0 B
website/build/blog/page/2.html 65.4 kB 0 B
website/build/blog/page/3.html 83.5 kB 0 B
website/build/blog/page/4.html 58.7 kB 0 B
website/build/blog/page/5.html 39.1 kB 0 B
website/build/blog/preparing-your-site-for-docusaurus-v3.html 138 kB -14 B (-0.01%)
website/build/blog/releases/2.2.html 49.6 kB -13 B (-0.03%)
website/build/blog/releases/2.3.html 60.8 kB -13 B (-0.02%)
website/build/blog/releases/2.4.html 65.3 kB -13 B (-0.02%)
website/build/blog/releases/3.0.html 111 kB -1 B (0%)
website/build/blog/releases/3.1.html 52 kB -12 B (-0.02%)
website/build/blog/releases/3.2.html 48.7 kB -13 B (-0.03%)
website/build/blog/releases/3.3.html 55.4 kB -13 B (-0.02%)
website/build/blog/releases/3.4.html 55.6 kB -6 B (-0.01%)
website/build/blog/tags.html 28.4 kB 0 B
website/build/blog/upgrading-frontend-dependencies-with-confidence-using-visual-regression-testing.html 128 kB -13 B (-0.01%)
website/build/docs.html 49.4 kB -4 B (-0.01%)
website/build/docs/advanced.html 31.1 kB -13 B (-0.04%)
website/build/docs/advanced/architecture.html 30.7 kB -9 B (-0.03%)
website/build/docs/advanced/client.html 76 kB -13 B (-0.02%)
website/build/docs/advanced/plugins.html 58.6 kB -13 B (-0.02%)
website/build/docs/advanced/routing.html 73.9 kB -5 B (-0.01%)
website/build/docs/advanced/ssg.html 81.5 kB -13 B (-0.02%)
website/build/docs/api/docusaurus-config.html 200 kB -13 B (-0.01%)
website/build/docs/api/misc/@docusaurus/eslint-plugin.html 48.3 kB -9 B (-0.02%)
website/build/docs/api/misc/@docusaurus/eslint-plugin/no-html-links.html 38.1 kB -13 B (-0.03%)
website/build/docs/api/misc/@docusaurus/eslint-plugin/no-untranslated-text.html 37 kB -13 B (-0.04%)
website/build/docs/api/misc/@docusaurus/eslint-plugin/prefer-docusaurus-heading.html 38.3 kB -9 B (-0.02%)
website/build/docs/api/misc/@docusaurus/eslint-plugin/string-literal-i18n-messages.html 41.9 kB -13 B (-0.03%)
website/build/docs/api/misc/@docusaurus/logger.html 39.8 kB -13 B (-0.03%)
website/build/docs/api/misc/create-docusaurus.html 34.9 kB -12 B (-0.03%)
website/build/docs/api/plugin-methods.html 67.4 kB -9 B (-0.01%)
website/build/docs/api/plugin-methods/extend-infrastructure.html 63 kB -9 B (-0.01%)
website/build/docs/api/plugin-methods/i18n-lifecycles.html 60.6 kB -9 B (-0.01%)
website/build/docs/api/plugin-methods/lifecycle-apis.html 170 kB -13 B (-0.01%)
website/build/docs/api/plugin-methods/static-methods.html 46.5 kB -9 B (-0.02%)
website/build/docs/api/plugins.html 31.6 kB -13 B (-0.04%)
website/build/docs/api/plugins/@docusaurus/plugin-client-redirects.html 62.6 kB 0 B
website/build/docs/api/plugins/@docusaurus/plugin-content-blog.html 172 kB -1 B (0%)
website/build/docs/api/plugins/@docusaurus/plugin-content-docs.html 202 kB -4 B (0%)
website/build/docs/api/plugins/@docusaurus/plugin-content-pages.html 78 kB -13 B (-0.02%)
website/build/docs/api/plugins/@docusaurus/plugin-debug.html 50.5 kB -13 B (-0.03%)
website/build/docs/api/plugins/@docusaurus/plugin-google-analytics.html 51.5 kB -13 B (-0.03%)
website/build/docs/api/plugins/@docusaurus/plugin-google-gtag.html 51 kB -17 B (-0.03%)
website/build/docs/api/plugins/@docusaurus/plugin-google-tag-manager.html 49.8 kB -13 B (-0.03%)
website/build/docs/api/plugins/@docusaurus/plugin-ideal-image.html 51.1 kB -9 B (-0.02%)
website/build/docs/api/plugins/@docusaurus/plugin-pwa.html 122 kB -9 B (-0.01%)
website/build/docs/api/plugins/@docusaurus/plugin-sitemap.html 68.5 kB -7 B (-0.01%)
website/build/docs/api/plugins/@docusaurus/plugin-vercel-analytics.html 42.2 kB -1 B (0%)
website/build/docs/api/themes.html 30.4 kB -13 B (-0.04%)
website/build/docs/api/themes/@docusaurus/theme-classic.html 46.4 kB -9 B (-0.02%)
website/build/docs/api/themes/@docusaurus/theme-live-codeblock.html 38.6 kB -9 B (-0.02%)
website/build/docs/api/themes/@docusaurus/theme-mermaid.html 37.5 kB -9 B (-0.02%)
website/build/docs/api/themes/@docusaurus/theme-search-algolia.html 34.7 kB -13 B (-0.04%)
website/build/docs/api/themes/configuration.html 256 kB -6 B (0%)
website/build/docs/blog.html 211 kB -1 B (0%)
website/build/docs/browser-support.html 51 kB -13 B (-0.03%)
website/build/docs/category/getting-started.html 28.3 kB 0 B
website/build/docs/category/guides.html 36.4 kB 0 B
website/build/docs/cli.html 63.8 kB -12 B (-0.02%)
website/build/docs/configuration.html 98.5 kB -9 B (-0.01%)
website/build/docs/create-doc.html 65.3 kB -1 B (0%)
website/build/docs/creating-pages.html 59.1 kB -12 B (-0.02%)
website/build/docs/deployment.html 211 kB -4 B (0%)
website/build/docs/docs-introduction.html 53.6 kB -13 B (-0.02%)
website/build/docs/docs-multi-instance.html 78.7 kB -9 B (-0.01%)
website/build/docs/docusaurus-core.html 245 kB -13 B (-0.01%)
website/build/docs/guides/whats-next.html 32.8 kB -13 B (-0.04%)
website/build/docs/i18n/crowdin.html 151 kB -1 B (0%)
website/build/docs/i18n/git.html 82.6 kB -9 B (-0.01%)
website/build/docs/i18n/introduction.html 50.7 kB -13 B (-0.03%)
website/build/docs/i18n/tutorial.html 171 kB 0 B
website/build/docs/installation.html 73 kB -1 B (0%)
website/build/docs/introduction/index.html 280 B 0 B
website/build/docs/markdown-features.html 83.2 kB -12 B (-0.01%)
website/build/docs/markdown-features/admonitions.html 118 kB -9 B (-0.01%)
website/build/docs/markdown-features/assets.html 93.5 kB -11 B (-0.01%)
website/build/docs/markdown-features/code-blocks.html 245 kB -9 B (0%)
website/build/docs/markdown-features/diagrams.html 56 kB +1 B (0%)
website/build/docs/markdown-features/head-metadata.html 52.8 kB -13 B (-0.02%)
website/build/docs/markdown-features/links.html 43.1 kB -13 B (-0.03%)
website/build/docs/markdown-features/math-equations.html 95.4 kB -2 B (0%)
website/build/docs/markdown-features/plugins.html 101 kB -5 B (0%)
website/build/docs/markdown-features/react.html 144 kB -13 B (-0.01%)
website/build/docs/markdown-features/tabs.html 148 kB -13 B (-0.01%)
website/build/docs/markdown-features/toc.html 88.8 kB -9 B (-0.01%)
website/build/docs/migration.html 43 kB -14 B (-0.03%)
website/build/docs/migration/v2.html 41.2 kB -13 B (-0.03%)
website/build/docs/migration/v2/automated.html 42.6 kB -13 B (-0.03%)
website/build/docs/migration/v2/manual.html 207 kB -13 B (-0.01%)
website/build/docs/migration/v2/translated-sites.html 52.8 kB -13 B (-0.02%)
website/build/docs/migration/v2/versioned-sites.html 67.2 kB -13 B (-0.02%)
website/build/docs/migration/v3.html 218 kB -12 B (-0.01%)
website/build/docs/playground.html 32.5 kB -13 B (-0.04%)
website/build/docs/resources/index.html 325 B 0 B
website/build/docs/search.html 121 kB -1 B (0%)
website/build/docs/seo.html 93 kB 0 B
website/build/docs/sidebar.html 136 kB -9 B (-0.01%)
website/build/docs/sidebar/autogenerated.html 156 kB 0 B
website/build/docs/sidebar/items.html 188 kB -13 B (-0.01%)
website/build/docs/sidebar/multiple-sidebars.html 67.1 kB -9 B (-0.01%)
website/build/docs/static-assets.html 56.2 kB -9 B (-0.02%)
website/build/docs/styling-layout.html 140 kB -10 B (-0.01%)
website/build/docs/support/index.html 319 B 0 B
website/build/docs/swizzling.html 120 kB +1 B (0%)
website/build/docs/team/index.html 310 B 0 B
website/build/docs/typescript-support.html 65.4 kB -13 B (-0.02%)
website/build/docs/using-plugins.html 115 kB 0 B
website/build/docs/versioning.html 87.1 kB 0 B
website/build/index.html 37.8 kB 0 B

compressed-size-action

@slorber slorber added the Argos Add this label to run UI visual regression tests. See argos.yml GH action. label Aug 1, 2024
Copy link

argos-ci bot commented Aug 1, 2024

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) ✅ No changes detected - Aug 1, 2024, 3:03 PM

@slorber
Copy link
Collaborator

slorber commented Aug 2, 2024

This does not lead to significant perf improvements according to benchmarks (https://github.com/OzakIOne/jsgitbench)

So let's just use execa for now (#10358)

@slorber slorber closed this Aug 2, 2024
@Josh-Cena Josh-Cena deleted the ozaki/simpleGit branch August 2, 2024 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Argos Add this label to run UI visual regression tests. See argos.yml GH action. CLA Signed Signed Facebook CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build fails on MacOS for large site because max proc limit is exceeded
3 participants