Replace gulp build with wp-scripts.

[peterwilsoncc]

Description of the Change

Replace gulp and various config files with @wordpress/scripts.

I think this is ready for review, it should probably get double sign-off as it is quite large. I recommend reviewing this with whitespace suppressed.

I have put together a second webpack config for the npm run release command as I think it is needed to copy files in to the release subdirectory for committing to the stable branch. As noted inline, I would love an alternative suggestion as it doesn't seem ideal.

Closes #892.

How to test the Change

• Ensure production file names remain the same
• Test the various builds/npm scripts have effectively the same result (ignoring whitespace, variable munging differences)

Changelog Entry

Changed - Build process now uses @wordpress/scripts in place of Gulp.

Credits

Props @peterwilsoncc, @dinhtungdu

Checklist:

• I agree to follow this project's Code of Conduct.
• I have updated the documentation accordingly.
• I have added tests to cover my change.
• All new and existing tests pass.

[peterwilsoncc]

I'm using this in place of the gulp copy script to copy all the files over during the release step.

I don't really like it or think it's a proper use of webpack so suggestions on how to do this without this would be much appreciated.

[peterwilsoncc]

I think I've addressed the various issues @dinhtungdu picked up:

• fc21fd1 fixes the block editor -- the svg needs to be a react component now
• 104ba23 fixes the name of the CSS file restoring admin-pull-table.min.css
• 4cd3fc8 fixes the external connections screen

WP Scripts doesn't seem to produce code compatible with older versions of WP so I've pulled in 04a1580 from the requirements bump PR as both the version bump and this issue are on the same milestone.

[dinhtungdu]

@peterwilsoncc Thanks for the updates! I can confirm all issues I noted are addressed.

I did another round of testing and found some additional issues, please check the screenshots below for more detail.

Screenshots

[peterwilsoncc]

Distributor panel restored in 0b53814 -- I'd reversed a media query.

For the checked attribute showing, I think that's expected due to the differences between properties and attributes.

The property shows the current state, the attribute the state at page load. If you run jQuery('#dt-as-draft').prop( 'checked' ) in the browser console, you should get an accurate result when unchecked.

[peterwilsoncc]

@dinhtungdu I think this is ready for another review. I recommend suppressing white space.

For now I have kept all the built asset file names the same which means the webpack config is a little odd. I've done this for back-compat but if it's decided it isn't needed then that can be done in a smaller follow up commit.

[dinhtungdu]

Awesome! 🎉
Thank you so much for this hard work! This PR tested well on my end.