Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated code to work with Grunt 1.0. #90

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Conversation

radum
Copy link

@radum radum commented May 17, 2016

I updated the code to make it work with the latest Grunt 1.0

Also Lodash is now a dependency as per Grunt best practices since 1.0

@radum radum mentioned this pull request May 17, 2016
@mikeapr4
Copy link

Apart from the NPM errors, looks good to me

@VeeTeeDev
Copy link

I did the same for the templating, but I moved the lodash to grunt itself because other dependencies are using it as well.

Can this be merged ? It's blocking :-)

@ba55ie
Copy link

ba55ie commented May 27, 2016

👍

@radum radum closed this Jun 1, 2016
@radum radum reopened this Jun 1, 2016
RaduM added 2 commits June 1, 2016 15:42
Update Travis and RequireJs dependency versions

Update Travis and RequireJs dependency versions
@radum
Copy link
Author

radum commented Jun 1, 2016

@mikeapr4 All good :D I was forgetting the Travis deps.

@VeeTeeDev
Copy link

@cloudchen

@radum
Copy link
Author

radum commented Jun 10, 2016

Ping @cloudchen :)

Should we clone this if is not maintained :(

@pksorensen
Copy link

what did you guys do ? cloned it to a new repository. no response from maintainers?

@radum
Copy link
Author

radum commented Oct 16, 2016

@pksorensen I'm using my own clone now. With the changes from this PR.

wpwood pushed a commit to wpwood/grunt-template-jasmine-requirejs that referenced this pull request Sep 7, 2017
@pranmitt
Copy link

pranmitt commented Dec 8, 2017

are you going to merge this ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants