Stop putting react-app-env.d.ts in src

[devuxer]

(This pertains to CRA w/ the --typescript flag.)

Putting a loose d.ts file at the top level of src adds clutter and breaks the convention of putting all d.ts files together in a folder, such as "types".

Suggestions to remedy this (in order of preference):

1. Honor the new location of react-app-env.d.ts if we move it.
2. Allow us to configure where this file goes.
3. Place the file in src/types.

---

EDIT:

@nhooyr Given its confusing, you could also add a comment to the generated file explaining why its there.

@bugzpodder This file was created to include create-react-app specific types. Removing it will cause errors like:

TypeScript error in ~/src/App.tsx(2,18):
Cannot find module './logo.svg'.  TS2307

    1 | import React from 'react';
  > 2 | import logo from './logo.svg';
      |                  ^
    3 | import './App.css';
    4 | 
    5 | const App: React.FC = () => {

This was created from #5611 if you want more details.

[bonavida]

I'm facing the same issue and I think @devuxer's request makes sense. Are you guys going to include some fix for this in the near future?

[cr4zyc4t]

I'm new to typescript and very excited about official support for TS of CRA.
Can anyone explain how should I use this file?

[KamilPacanek]

Another question from me: what is the reason behind this file? I've found out that it is generated upon starting dev server. Should I .gitignore it?

[ktalebian]

+1 for this; I don't understand why it is generated, and I've added it to my .gitignore

[henriquepw]

I fully agree

[robinvdv12]

I don't see the point of this file and can't find any usefull information on the internet. +1 for this issue.

[bugzpodder]

It contains types for the bootstrapped App. Removing it will cause errors like:

TypeScript error in ~/src/App.tsx(2,18):
Cannot find module './logo.svg'.  TS2307

    1 | import React from 'react';
  > 2 | import logo from './logo.svg';
      |                  ^
    3 | import './App.css';
    4 | 
    5 | const App: React.FC = () => {

This was created from #5611 if you want more details.

[bugzpodder]

I don't really see this as a blocker for anyone. Having an extra file isn't ideal but it's pretty minimal by itself.

[bonavida]

So, it's not possible to run the bootstrapped app (with typescript) without creating this file? Maybe you should let people choose where to have it. What about tsconfig.json?

[devuxer]

@bugzpodder, So, one of the criteria for an issue is that it has to be a "blocker"? Perhaps that should be noted when people actually go to submit an issue, so we don't waste our own and everyone else's time.

[iansu]

@devuxer No, being a "blocker" is not a requirement for submitting an issue. However, following the code of conduct and being respectful to people is.