Skip to content

Update build scripts for new process#11801

Merged
turt2live merged 8 commits intotravis/sourcemapsfrom
travis/babel7-wp-scripts
Jan 9, 2020
Merged

Update build scripts for new process#11801
turt2live merged 8 commits intotravis/sourcemapsfrom
travis/babel7-wp-scripts

Conversation

@turt2live
Copy link
Copy Markdown
Member

@turt2live turt2live commented Jan 8, 2020

This is against travis/sourcemaps for safety.

Note: we needed to convert something to TypeScript so it doesn't complain that there's nothing to compile, so the smallest utility file was chosen.

This is based on #11712 - see pull/11712/head...pull/11801/head for the real diff

Many of the script changes (including linters) are pulled in from the react-sdk.

This moves the babel and postcss configs into the webpack config for ease of maintenance (and because we need variations of them). The typescript config is left outside the webpack config for IDEs to pick it up.
For reasons described within this diff.
We have to convert *something* to TypeScript so it doesn't complain that there's nothing to compile, so this converts the easiest utility library.

Many of the scripts are copied from the react-sdk.
@turt2live
Copy link
Copy Markdown
Member Author

A failing build isn't terribly surprising for this PR - 2/3rds of the dependencies are missing/not mapped.

@turt2live turt2live requested a review from a team January 8, 2020 23:55
@turt2live turt2live merged commit a91bab2 into travis/sourcemaps Jan 9, 2020
@turt2live turt2live deleted the travis/babel7-wp-scripts branch January 9, 2020 20:34
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.

2 participants