Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Debug build #137

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

lasseborly
Copy link
Contributor

@lasseborly lasseborly commented May 25, 2021

Outputs the production build to /dist/production and the debug build to /dist/debug.

This enables easier debugging at runtime and allows for the usage of tools like React and Redux dev tools.

@lasseborly
Copy link
Contributor Author

!!OBS!! this directory structure is probably not backwards compatible with the current consumers.

@lasseborly lasseborly marked this pull request as ready for review June 2, 2021 11:59
@lasseborly
Copy link
Contributor Author

This is ready for review @kasperg

@lasseborly
Copy link
Contributor Author

To remain backwards compatible we could output the production build to /dist and have the debug build be as stated in /dist/debug. Further down the line we could then also have /dist/performance etc.

The implicity rubs me the wrong way though.

@lasseborly lasseborly force-pushed the chore/debug-build branch from d5b5ca5 to 6e88cc6 Compare June 2, 2021 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant