Do not override any .env settings in .gitignore files

[petebacondarwin]

Previously we only looked for .env* in the gitignore but now we cover more cases such as:

• .env
• .env\*
• .env.<local|production|staging>
• .env\*.<local|production|staging>

Fixes #10749

---

• Tests
  • Tests included
  • Tests not necessary because:
• Public documentation
  • Cloudflare docs PR(s):
  • Documentation not necessary because: bug fix
• Wrangler V3 Backport
  • Wrangler PR:
  • Not necessary because: create-cloudflare not backported

[changeset-bot]

🦋 Changeset detected

Latest commit: 293edad

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[pkg-pr-new]

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@10766 

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@10766 

miniflare

npm i https://pkg.pr.new/miniflare@10766 

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@10766 

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@10766 

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@10766 

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@10766 

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@10766 

wrangler

npm i https://pkg.pr.new/wrangler@10766 

commit: 293edad

[vicb]

The commit & changeset say:

do not override .env settings in .gitignore files

but IIUC this is an existing feature that is being fixed by updating the regexp?
If that is the case, could we update the decsriptions to reflect this?

[petebacondarwin]

The commit & changeset say:

do not override .env settings in .gitignore files

but IIUC this is an existing feature that is being fixed by updating the regexp? If that is the case, could we update the decsriptions to reflect this?

Done

[vicb]

LGTM thanks for the update (two inline typo fixes)

Should <local|production|staging> be <local|production|staging|...> instead?

i.e. it could be "test" or anything else