[docs] Make command work on both Linux and Windows

[Zenahr]

Change command to backslash so it works on both Linux and Windows machines. Trying to run the command as is results in a syntax error on Windows.

Pre-flight checklist

• I have read the Contributing Guidelines on pull requests.
• If this is a code change: I have written unit tests and/or added dogfooding pages to fully verify the new behavior.
• If this is a new API or substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Motivation

Test Plan

Test links

Deploy preview: https://deploy-preview-_____--docusaurus-2.netlify.app/

Related issues/PRs

#8703

[Zenahr]

Correction: the command cp -r docs\** i18n\fr\docusaurus-plugin-content-docs\current doesn't work on Windows.
I'm getting a cp: cannot stat 'docs**': No such file or directory error.

[netlify]

✅ [V2]

Built without sensitive environment variables

Name	Link
🔨 Latest commit	4aeae34
🔍 Latest deploy log	https://app.netlify.com/sites/docusaurus-2/deploys/63fc78449fbbbe00080659db
😎 Deploy Preview	https://deploy-preview-8704--docusaurus-2.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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

[github-actions]

⚡️ Lighthouse report for the deploy preview of this PR

URL	Performance	Accessibility	Best Practices	SEO	PWA	Report
/	🟠 85	🟢 97	🟢 100	🟢 100	🟢 90	Report
/docs/installation	🟠 73	🟢 100	🟢 100	🟢 100	🟢 90	Report

[Zenahr]

We might want to add a small text describing what the code snippet does too. Just a thought.

[slorber]

We could use tabs if you can find another command that do the same for windows?

[Zenahr]

We could use tabs if you can find another command that do the same for windows?

Yep, I was thinking that. I'll add that in.

[Zenahr]

@slorber my PR LGTM now. The commands now show up in tabs and the Windows commands work on Windows.

[Zenahr]

Related to #8703

[Zenahr]

One note on the Xcopy docs i18n\fr\docusaurus-plugin-content-docs\current /E /C /I /D command: This opens a prompt for all files. I haven't yet found a flag for declining (no to all). /Y exists, for accepting all prompts, which we don't want here though I think as this would replace all already translated files.

We might have to separate the translation workflow out into first time and continued translations.

[slorber]

That looks a bit complex just to copy files 😅

What about using a nodejs package that works the same on both platforms?

npx cpy-cli 'docs/**' 'i18n/fr/docusaurus-plugin-content-docs/current'

https://github.com/sindresorhus/cpy-cli

[Zenahr]

@slorber I agree...
I'll look at cpy-cli and update the PR. Thanks for the suggestion!

[armano2]

this command is working fine on windows as long as you use powershell instead of cmd

tested in PowerShell 7.3.4

https://learn.microsoft.com/en-us/powershell/

[Zenahr]

I agree with Sebastién that a single, cross-platform command would be best. See here: #8704 (review)

[AshwanthramKL]

Hey @Zenahr, is there anyway I can help with this?

[slorber]

closing until someone comes up with a simple PR

[0xSaksham]

@slorber I believe this issue is closed for a simple PR, should I work on it?

[slorber]

Not sure what you mean @0xSaksham

you can submit a new PR taking into consideration my existing review on this PR