docs(i18n): fix cp commands to avoid duplicate files#11634
docs(i18n): fix cp commands to avoid duplicate files#11634Rudra-Tiwari-codes wants to merge 1 commit intofacebook:mainfrom
Conversation
The cp commands in the i18n tutorial were using both -r (recursive) AND ** (glob pattern), which causes duplicate files in some shells where the glob expands all files before the recursive copy runs. Changes: - Replace docs/** with docs/. in i18n-tutorial.mdx - Replace blog/** with blog/. in i18n-tutorial.mdx - Replace docs/** with docs/. in i18n-git.mdx - Replace blog/** with blog/. in i18n-git.mdx The /. syntax copies the contents of the directory recursively without relying on shell globbing, avoiding the duplication issue. Fixes #11158
✅ [V2]Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Warning: This author is forking multiple ML projects such as google-deepmind/alphafold, ml-explore/mlx, openai/CLIP, pytorch/pytorch, tensorflow/tensorflow, anthropics/claude-code, vllm-project/vllm, and others, adding minimal "contributions" (often for tests or miscellaneous changes) without proper validation. A review of their commits shows mostly local implementations of TODOs copied from existing projects, with little to no substantive review or testing. So far, this author has forked 41 repositories following the same pattern. Be careful when accepting this PR. It’s also concerning how this author is able to submit PRs across four repositories in the same day, each requiring large context, which strongly suggests a highly automated workflow. |
|
Thank you for the feedback regarding my recent activity. I am a student and I have been using these smaller tasks as a way to familiarize myself with the architecture of various codebases. I was not aware that submitting multiple minor pull requests was considered disruptive to the maintainer workflow or seen as contribution padding. I appreciate the correction and will pay much closer attention to the impact of my work moving forward. I am closing this pull request now to focus on delivering more substantive technical contributions that provide genuine value to the community. |
Pre-flight checklist
Motivation
Fixes #11158
The
cpcommands in the i18n tutorial use both-r(recursive) AND**(glob pattern), which causes duplicate files in some shells where the glob expands all files in subdirectories before the recursive copy runs.For example,
cp -r docs/**results in:Instead of just the directory structure.
Test Plan
cp -r docs/.is POSIX-compliant and works correctly/.syntax copies directory contents recursively without relying on shell globbingTest links
Deploy preview: https://deploy-preview-____--docusaurus-2.netlify.app/docs/i18n/tutorial#translate-markdown-files
Related issues/PRs
Closes #11158
Note: Previous PRs #11450 and #11471 attempted to fix this but were closed because the contributors did not sign the CLA.