Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

PLAT-307 update readme grunt command #325

Merged
merged 4 commits into from
Jun 28, 2016
Merged

Conversation

gusliedke
Copy link
Contributor

@bimsonz bimsonz added this to the RND17 September milestone Jun 14, 2016
@@ -97,8 +97,11 @@ Grunt will concatenate all js files in /scripts and minify them to /js

#####In rnd17

grunt rnd17 --target=themes/rnd17/config.rb
go to /profile/cr, and
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gusliedke why do we need to cd to /profile/cr then target ../../ does doing it from the root not work ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bimsonz you need to be where gruntfile.js is to run the task

@bimsonz
Copy link
Contributor

bimsonz commented Jun 22, 2016

@gusliedke do you need to be watching both the themes at once or is having grunt rnd17 OR grunt campaign_base running once at at time ok ?

I've made some changes, can you review and test these ? you should just be able to run phing grunt:watch and it will watch the theme thats relevant to the site youre on (but on rnd17 this wont watch the campaign theme as well as the rnd17 theme, it will be just rnd17) that coooool?

@gusliedke
Copy link
Contributor Author

@bimsonz That sounds right to me. I will test it asap

@gusliedke gusliedke merged commit af3fd45 into develop Jun 28, 2016
@gusliedke gusliedke deleted the feature/PLAT-307_updoc branch June 28, 2016 09:24
@pvhee pvhee mentioned this pull request Jan 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants