chore: ignore next-env.d.ts

[skuridin]

✅ Checklist

• I have followed every step in the contributing guide (updated 2022-10-06).
• The PR title follows the convention we established conventional-commit
• I performed a functional test on my final commit

---

Changelog

Added next-env.d.ts to .gitignore as recommended in Next.js documentation.

---

Screenshots

💯

[changeset-bot]

🦋 Changeset detected

Latest commit: faa2821

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

This PR includes changesets to release 1 package

Name	Type
create-t3-app	Patch

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

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
create-t3-app	✅ Ready (Inspect)	Visit Preview	Nov 18, 2022 at 11:55PM (UTC)

[juliusmarminge]

Then we can remove this part in the docs maybe?

https://create.t3.gg/en/folder-structure#next-envdts

cc @nexxeln

[nexxeln]

Pretty sure the docs are incorrect here. This file is required for TypeScript to work in Next.js and it should not be removed or edited. So if someone clones a project they would have to make this file on their own.

In the latest create-next-app as well this is added and committed in the initial commit.

[juliusmarminge]

Pretty sure the docs are incorrect here. This file is required for TypeScript to work in Next.js and it should not be removed or edited. So if someone clones a project they would have to make this file on their own.

In the latest create-next-app as well this is added and committed in the initial commit.

Please make this PR to the Next.js repo and fix their docs haha.

It's generated everytime you run dev or build

[nexxeln]

Pretty sure the docs are incorrect here. This file is required for TypeScript to work in Next.js and it should not be removed or edited. So if someone clones a project they would have to make this file on their own.
In the latest create-next-app as well this is added and committed in the initial commit.
Please make this PR to the Next.js repo and fix their docs haha.

It's generated everytime you run dev or build

Oh I didn't know that. But then why does create-next-app put it in initial commit?

[skuridin]

Oh I didn't know that. But then why does create-next-app put it in initial commit?

I just created new create-next-app and while the file is present, it is not in the "Initial commit from Create Next App" git commit.

I can only assume it's copied to the new project to avoid surprises from sudden file appearance.

[nexxeln]

Oh I didn't know that. But then why does create-next-app put it in initial commit?

I just created new create-next-app and while the file is present, it is not in the "Initial commit from Create Next App" git commit.

I can only assume it's copied to the new project to avoid surprises from sudden file appearance.

Ah okay that makes sense.

[nexxeln]

Then we can remove this part in the docs maybe?

https://create.t3.gg/en/folder-structure#next-envdts

cc @nexxeln

It's still a file that is being generated, so I think we should keep it

[nexxeln]

LGTM

[github-actions]

⚡️ Lighthouse report for the changes in this PR:

Category	Score
🟢 Performance	100
🟢 Accessibility	100
🟢 Best practices	100
🟠 SEO	86
🟠 PWA	54

Lighthouse ran on https://create-t3-app-git-fork-skuridin-chore-next-env-gitignore-t3-oss.vercel.app/