Skip to content
This repository has been archived by the owner on Jun 14, 2022. It is now read-only.

Improve dev tooling configuration #61

Closed
3 of 4 tasks
mindctrl opened this issue Feb 25, 2019 · 0 comments
Closed
3 of 4 tasks

Improve dev tooling configuration #61

mindctrl opened this issue Feb 25, 2019 · 0 comments
Assignees
Labels
Milestone

Comments

@mindctrl
Copy link
Member

mindctrl commented Feb 25, 2019

Right now things aren't ideal for multiple devs working in parallel. Since scripts and styles are always compiled down to a single file, even during dev, it's next to impossible to work in parallel due to merge conflicts.

At a minimum we need to compile individual block scripts and styles to separate files and load those when SCRIPT_DEBUG and/or WP_DEBUG is true. Then we can combine them prior to release using the build tools.

  • phpcs
  • break out of create-guten-block
  • transpile and load block JS/CSS files separately in debug mode
  • remove bundled scripts from this repo and only deliver them in releases
@mindctrl mindctrl added the chore label Feb 25, 2019
@mindctrl mindctrl added this to the 1.5.3 milestone Feb 25, 2019
@mindctrl mindctrl self-assigned this Feb 25, 2019
mindctrl added a commit that referenced this issue Feb 25, 2019
mindctrl added a commit that referenced this issue Feb 25, 2019
@mindctrl mindctrl modified the milestones: 1.5.3, 1.5.4, 1.5.5 Feb 26, 2019
mindctrl added a commit that referenced this issue Mar 4, 2019
mindctrl added a commit that referenced this issue Mar 4, 2019
mindctrl added a commit that referenced this issue Mar 5, 2019
mindctrl added a commit that referenced this issue Mar 5, 2019
Adds node-wp-i18n for building .pot file. Adds 'node-makepot' npm script.

Adds npm-run-all for running multiple scripts sequentially.

Moves existing 'build' script to 'build-assets'.

Updates 'build' script to run node-makepot and build-assets scripts using npm-run-all.
mindctrl added a commit that referenced this issue Mar 5, 2019
mindctrl added a commit that referenced this issue Mar 5, 2019
mindctrl added a commit that referenced this issue Mar 5, 2019
* Add phpcs dependencies. #61

* Add phpcs scripts to composer.json. #61
mindctrl added a commit that referenced this issue Mar 5, 2019
mindctrl added a commit that referenced this issue Mar 5, 2019
Update build script to use wp i18n.

Remove unused grunt-wp-i18n package.
mindctrl added a commit that referenced this issue Mar 5, 2019
* Eject from create-guten-block. #61

* Update .gitignore. #61

* Update build script. #61.

Adds npm-run-all for running multiple scripts sequentially.

Moves existing 'build' script to 'build-assets'.

Updates 'build' script to run node-makepot and build-assets scripts using npm-run-all.

* Remove JS and CSS build files. #61

Update build script to use wp i18n.

Remove unused grunt-wp-i18n package.
@mindctrl mindctrl closed this as completed Mar 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant