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

Feature/plat 229 gruntjs update #246

Merged
merged 15 commits into from
May 18, 2016
Merged

Conversation

gusliedke
Copy link
Contributor

@gusliedke
Copy link
Contributor Author

@bimsonz it's bullet proof!
New Grunt tasks

In campaign
    `grunt campaign_base`
    `grunt build`

In rnd17
    `grunt rnd17 --target=themes/rnd17/config.rb`
    `grunt build_all`

@gusliedke
Copy link
Contributor Author

Can we make build.xml to run grunt build in Campaign repo and grunt build_all in Rnd17 ?

@bimsonz
Copy link
Contributor

bimsonz commented May 17, 2016

@gusliedke So, to keep the gruntfile with the profile ive moved it back and updated a few things.. please review this and let me know if youre happy and ill merge it!

we can add <property name="grunt.cmd" override="true" value="build"/>
to the rnd17 build.xml build.properties to override the value to build_all and this should be done :)

@bimsonz bimsonz merged commit c7852ed into develop May 18, 2016
@bimsonz bimsonz deleted the feature/PLAT-229_gruntjs_update branch May 18, 2016 09:28
@bimsonz bimsonz self-assigned this May 18, 2016
@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