Skip to content
This repository has been archived by the owner on Apr 25, 2023. It is now read-only.

Typescript aliases #11

Closed
simenandre opened this issue Sep 16, 2019 · 3 comments
Closed

Typescript aliases #11

simenandre opened this issue Sep 16, 2019 · 3 comments
Labels
help wanted Extra attention is needed

Comments

@simenandre
Copy link
Member

Should we use typescript aliases? The benefits are removing the ../../../../../ problem. In the architecture that is played out with Stact, we will have this problem more often.

I suggest that we add the following:

{
  "compilerOptions": {
   ...
    "paths": {
      "@modules/*": ["src/modules/*"],
      "@generated/*": ["src/generated/*"],
      "@store/*": ["src/store/*"]
    }
  }
}

Reference:

@Eullin
Copy link

Eullin commented Sep 17, 2019

I don't see any cons, so I think we should go for it.

@hexjelly hexjelly added the help wanted Extra attention is needed label Oct 2, 2019
@hexjelly
Copy link
Contributor

hexjelly commented Oct 2, 2019

This does not seem to be possible together with react-scripts for some reason: facebook/create-react-app#5645

@simenandre
Copy link
Member Author

Okey, since it's not possible. We'll keep it as is.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants