[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