I found an old PR from 2016 that reduced the size to what it is today.
https://github.com/apache/incubator-superset/pull/1679

Seems like there is no harm deleting it as the reasoning is weakened today with 
RAMs getting bigger. 😄  

We should probably update this bit in `CONTRIBUTING.md` before deleting `prod` 
and also make a note in `UPDATING.md` in case anyone has `npm/yarn run prod` in 
their build pipeline.
```
# Compile the Javascript and CSS in production/optimized mode for official 
releases
npm run prod
```
Thoughts? @mistercrunch @williaster @kristw 

I'm also cool with this PR shipping as-is. Going to go ahead and approve now. 

[ Full content available at: 
https://github.com/apache/incubator-superset/pull/5934 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to