-
Notifications
You must be signed in to change notification settings - Fork 321
Confused on what is happening on your configs #22
Comments
I'm in agreement. I'm more in favor of removing the dynamic features from these configs. There should be one default config and each derivation directly overrides the configuration options. It just takes forever to read and understand what's going on. |
Does this I agree the config files are a beast. But i also consider this the ultimate webpack example, by the webpack author actually. I'd rather learn from it than have it "dumbed down". There are plenty of simple webpack examples. |
👍 Code duplication in config files is absolutely normal. Much more better than imperative instructions. Configs are meant to be compared with diff tools to inspect the difference. Here we have basically 3 presets: |
@ivan-kleshnin Care to share your refactor? |
@pikeas sure, here is raw a version https://github.com/Paqmind/starter. A newer variation of it is used here https://github.com/Paqmind/react-ultimate |
Some are straight forward, but some are not, especially on production, I don't know where is the index file.. I have to resort to reading source code for learning, waaah, I hope the react-starter will improve.
Was there anyway, that you don't have to build then deploy? I can see that you have to run a command, followed by another?
Also, if I weren't using express-node for launching server, wouldn't it be good, if on production, there's a dist file, were index.html lives there, along with its resources?
The text was updated successfully, but these errors were encountered: