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

Add grunt-contrib-connect as simple server #40

Closed
castus opened this issue Jul 14, 2014 · 8 comments
Closed

Add grunt-contrib-connect as simple server #40

castus opened this issue Jul 14, 2014 · 8 comments

Comments

@castus
Copy link

castus commented Jul 14, 2014

Hi Lubos,

I have one simple feature request. Could you add grunt-contrib-connect to package.json and simple task "server" to Gruntfile.js? Each time I set up the project, need to add server to run it on local machine. This improvement should speed up project set up. What do you think?

@luboskmetko
Copy link
Member

Hey Krzysztof,

maybe this could be optional? Because we assume most developers have their own web servers already set up and running.

What you think?

Thanks

@castus
Copy link
Author

castus commented Jul 15, 2014

Yes, I thinking of it as separate task in Gruntfile. It can be “grunt server” to run simple server with index.html as base. I just want to have dependencies in package.json and simple configuration in Gruntfile.js to not add it by hand every time. Can be done?

THanks,
KR


Z optymizmem,
Krzysztof Romanowski
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)

On Tuesday, 15 July 2014 at 10:21, Lubos Kmetko wrote:

Hey Krzysztof,
maybe this could be optional? Because we assume most developers have their own web servers already set up and running.
What you think?
Thanks


Reply to this email directly or view it on GitHub (#40 (comment)).

@castus
Copy link
Author

castus commented Jul 15, 2014

I found small issue, I have this error, while trying to create CSS:

Running "sass:dist" (sass) task
OptionParser::InvalidOption: invalid option: --sourcemap
Use --trace for backtrace.
Warning: Exited with error code 1 Used --force, continuing.

I have Sass 3.2.14 (Media Mark)


Z optymizmem,
Krzysztof Romanowski
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)

On Tuesday, 15 July 2014 at 10:21, Lubos Kmetko wrote:

Hey Krzysztof,
maybe this could be optional? Because we assume most developers have their own web servers already set up and running.
What you think?
Thanks


Reply to this email directly or view it on GitHub (#40 (comment)).

@castus
Copy link
Author

castus commented Jul 15, 2014

And small improvement. Grunt “watch" task don’t re-run if you create folder in scss and place files there. This is because “watch” only watching .scss files in /scss folder, not in folders within it.
To fix it, instead of
scss: {
files: ['<%= xh.src %>/scss/
.scss'],
},

add “/” as below:
scss: {
files: ['<%= xh.src %>/scss/
/*.scss'],
},

Than, each *.scss file will be watched by grunt task.
Hope it helps ;)

KR


Z optymizmem,
Krzysztof Romanowski
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)

On Tuesday, 15 July 2014 at 10:21, Lubos Kmetko wrote:

Hey Krzysztof,
maybe this could be optional? Because we assume most developers have their own web servers already set up and running.
What you think?
Thanks


Reply to this email directly or view it on GitHub (#40 (comment)).

@luboskmetko
Copy link
Member

Hey Krzysztof,

by optional I meant that the generator would ask user if they want to create a server and only then would set up appropriate package and task in Gruntfile.

Regarding the sourcemap issue, can you please upgrade your Sass version, as source maps are supported only since version 3.3.0

Thanks for the improvement suggestion, I'll create a separate issue from it and we will implement it.

@castus
Copy link
Author

castus commented Jul 16, 2014

Hey Lubos,

Sounds very good, can’t wait until it will be implemented in that way :)

THanks,
KR


Z optymizmem,
Krzysztof Romanowski
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)

On Wednesday, 16 July 2014 at 13:21, Lubos Kmetko wrote:

Hey Krzysztof,
by optional I meant that the generator would ask user if they want to create a server and only then would set up appropriate package and task in Gruntfile.
Regarding the sourcemap issue, can you please upgrade your Sass version, as source maps are supported only since version 3.3.0
Thanks for the improvement suggestion, I'll create a separate issue from it and we will implement it.


Reply to this email directly or view it on GitHub (#40 (comment)).

@piotrkulpinski
Copy link
Contributor

Fixed in #31

@castus
Copy link
Author

castus commented Aug 12, 2014

Thanks guys!

@yuijim yuijim closed this as completed Aug 13, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants