Closed mattsherman closed 2 months ago
Tested and reviewed the code. Was able to build and run the extension successfully.
Thanks!
I think you need to add the issue to a milestone. Can you please do that?
Done (this public repo didn't have any milestones, but there is no reason to not start using them! 😄)
Looking good, @mattsherman! Nice catch on the unused prop!
Before approving, how about adding a similar
.nvmrc
file as well?
Great idea -- done in d8685c7!
Description
This PR updates the dev dependencies and build scripts, allowing this project to be successfully built in the same dev environment as other extensions such as the base WooCommerce Product Recommendations.
Closes #9
Changes included
devDependencies
updated to be the same as the WooCommerce Product Recommendations reponode-sass
replaced withsass
autoprefixer
'sbrowsers
option changed tooverrideBrowserslist
to avoid deprecation notice@somewherewarm/selectsw
dependency removed, as it is unused in this repoTesting instructions
On a system with WooCommerce Product Recommendations installed...
npm i
- verify dependencies are installed without error (used to complain about missingpython2
)grunt build
- verify build completes without errorassets/css/admin/wc-prl-cl-admin.css
assets/js/admin/wc-prl-cl-admin.js