[wp-trac] [WordPress Trac] #51966: npm/grunt watch/build task names are inconsistent and unintuitive
WordPress Trac
noreply at wordpress.org
Thu Mar 4 05:55:49 UTC 2021
#51966: npm/grunt watch/build task names are inconsistent and unintuitive
-----------------------------------+-------------------------------
Reporter: iandunn | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Build/Test Tools | Version: 5.1
Severity: normal | Resolution:
Keywords: has-patch 2nd-opinion | Focuses: javascript, docs
-----------------------------------+-------------------------------
Comment (by isabel_brison):
> Remove the build/ folder entirely.
Do any dev environment setups rely on this folder currently? Would be good
what its original purpose was before removing it.
Also, something to consider given WP keeps moving in a more JS-heavy
direction is that JS projects typically are structured with separate `src`
and `build` folders where only `src` is version controlled and `build`
generated as needed for production packages or local development (there
may be different versions for each, e.g. the development build will likely
include sourcemaps that won't be added to the production build). Going in
a completely different direction to this will be confusing to JS devs
looking to contribute to the project.
> We'd setup a A GitHub action to automatically compile & commit the built
files to the repo on a post-push hook, like @ryelle did for
TwentyTwentyOne.
I'm probably missing something obvious, but why do we need to commit built
files to the repo?
> I have a feeling that zero-installs fit better with WordPress's
philosophies. Zero-installs are decidedly more accessible as they require
less tooling on the part of a new contributor.
This is definitely an interesting idea, but I suspect the biggest hurdle
for new contributors is getting their local environment set up, and
running a build script would be easy in comparison. It would be great to
have some data on this though!
--
Ticket URL: <https://core.trac.wordpress.org/ticket/51966#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list