[buddypress-trac] [BuddyPress Trac] #7551: Merge Nouveau template directory to BP core

buddypress-trac noreply at wordpress.org
Sat Sep 9 10:10:12 UTC 2017


#7551: Merge Nouveau template directory to BP core
-------------------------+------------------
 Reporter:  hnla         |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  3.0
Component:  Templates    |     Version:
 Severity:  normal       |  Resolution:
 Keywords:               |
-------------------------+------------------

Comment (by netweb):

 Replying to [comment:18 hnla]:
 > @netweb when I originally tested nouveau directory copied out to run
 under core bp-templates I re-built bp core gruntfile locally so my working
 environment would work or at least tested to see if could work. Merged
 your last change and pushed up patch with my watch task and lint sections,
 if it helps.

 Thanks for the patch, I'll update and merge it into the patch for #6382 so
 that we can watch _all the things_

 > One thing that occurred to me though is in the task 'src' we include
 'sass' even though we would have had to compile sass previously so maybe
 we don't need to include that just keep to the lint, jshint, rtl etc?

 I'll take a closer look at that also

 We're also currently "shipping" the SCSS files in the `build/` folder
 which I'll also fix

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7551#comment:20>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list