WordPress.org
Get WordPress

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 5 years ago

#46706 closed defect (bug) (fixed)

Ignore Twenty Nineteen's node_modules directory in "jsvalidate:build" task

Reported by: SergeyBiryukov Owned by: SergeyBiryukov
Priority: normal Milestone: 5.2
Component: Build/Test Tools Version:
Severity: normal Keywords:
Cc: Focuses:

Description (last modified by SergeyBiryukov)

Twenty Nineteen uses npm to run various build tasks: node-sass, rtlcss, etc.

Once you run npm install in the theme's directory, grunt build core task fails due to jsvalidate:build attempting to validate files in twentynineteen/node_modules. This directory should be excluded from the task.

Change History (5)

#1 @SergeyBiryukov
7 years ago

  • Description modified (diff)

#2 @SergeyBiryukov
7 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 45060:

Build/Test Tools: Exclude twentynineteen/node_modules from jsvalidate:build task.

Fixes #46706.

#3 @SergeyBiryukov
7 years ago

  • Component Bundled Theme → Build/Test Tools

#4 @SergeyBiryukov
7 years ago

In 47348:

Built/Test Tools: Ignore the node_modules folder in Twenty Nineteen when running grunt jshint:themes.

Follow-up to [45060], [46352], [46450].

Fixes #49494. See #46706.

#5 @desrosj
5 years ago

#46099 was marked as a duplicate.

Note: See TracTickets for help on using tickets.