Skip to content
This repository has been archived by the owner on Jun 3, 2019. It is now read-only.

Rename 'shared' folder to 'src' folder #521

Open
oyeanuj opened this issue Oct 7, 2017 · 4 comments
Open

Rename 'shared' folder to 'src' folder #521

oyeanuj opened this issue Oct 7, 2017 · 4 comments

Comments

@oyeanuj
Copy link
Contributor

oyeanuj commented Oct 7, 2017

Hi folks! I was wondering if in the next release, we should rename shared folder to src folder. Mostly because it increasingly seems to be the convention (create-react-app popularizes it), and lot of other tools seem to be trying to work out of the box with that convention.

For example, I was trying to integrate catalog as a living style guide for my project and ran into issues. And as per one of the members of the catalog project, they try to work out of the box for projects where components live in src. This is also true for others like react-styleguidist.

Has this been discussed already before? Thoughts?

@SleepWalker
Copy link

I'm voting for app. Because we have source files in server, client, config too. With additional tweaks you can achieve something like this:

import config from 'config';
import Component from 'app/components/Component';

Which looks really nice

For example, I was trying to integrate catalog as a living style guide for my project and ran into issues. And as per one of the members of the catalog project, they try to work out of the box for projects where components live in src. This is also true for others like react-styleguidist.

This is an issue of catalog. You can use the tool, that does not make assumptions about the project it runs in, e.g. storybook.

@oyeanuj
Copy link
Contributor Author

oyeanuj commented Oct 17, 2017

@SleepWalker I'm open to app as well if that is picking up as a convention. Alternatively, we could think of app being under src which is also a prevalent convention.

To be clear, the matter isn't about catalog, but the fact that as tools that pick a configuration to default to.. we should embrace it as well. create-react-app is the a big way this convention is set, and I just gave the examples of couple of tools that work out of the box for CRA apps, mainly due to the file structure.

@Falieson
Copy link

Falieson commented Nov 13, 2017

  1. I agree that shared should be src
  2. I also think shared/utils/ should be shared/helpers
  3. And that shared/Html/index.js should be shared/helpers/renderHtml.js

From my shared/README.md

.
├── /app/                             # The Application
    ├── /layouts/                     # For different screen resolutions
    └── /routes/                      # 
├── /helpers/                         # 
├── /modules/                         # 
└── /redux/                           # 

@Falieson
Copy link

Falieson commented Nov 15, 2017

The base is my typescript Fork, but should be like 99% of the work if someone wants to copy it:
Falieson/2017-typescript-react@c78d994

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants