These are chat archives for jescalan/roots

1st
Jun 2015
Henry Snopek
@hhsnopek
Jun 01 2015 00:16
You're going to want to use like RequireJS to require the files into your main.coffee
then you can use roots-browserify to concat all of them together
that should do the trick
I would create a roots-extension for this though, it would be really useful
Daniel Box
@dbox
Jun 01 2015 00:48
@hhsnopek Thanks! I was hoping the "./" before bower_components would do the trick as It seems like the one thing i haven't tried, yet but it's still popping the folder in public no matter what i do...
module.exports =
  ignores: ['bower*', './bower_components/**', 'readme.md', '**/layout.*', '**/_*', '.gitignore', 'ship.*conf']
This gets rid of bower.json, but not the bower_components folder ...
Daniel Box
@dbox
Jun 01 2015 00:55
Also isn't roots concatting them all together for me anyway?
  extensions: [
    js_pipeline
      manifest: 'assets/js/js-manifest.yml'
      out: 'js/build.js'
      minify: true
Henry Snopek
@hhsnopek
Jun 01 2015 01:27
bower_components/**
that works
I cloned your project and tested it
and you're right, I didn't realize what the manifest was for until now haha
learn something new everyday
Daniel Box
@dbox
Jun 01 2015 01:33
Yeah that works to hide it, but it also strips out all the css and js
I just dont understand why it needs the bower_components in the public to work. it's not eevn using any of the files. its using the concatted js and css
Henry Snopek
@hhsnopek
Jun 01 2015 01:35
well putting it into the ignores array ignores it completely from any roots process I believe
Daniel Box
@dbox
Jun 01 2015 01:36
so basically I can't use out of the box roots tools for what i want then, correct?
Henry Snopek
@hhsnopek
Jun 01 2015 01:37
hmm give me a second
Daniel Box
@dbox
Jun 01 2015 01:38
thanks
@hhsnopek added you as a collaborator if you want to to push up w/e you have
Henry Snopek
@hhsnopek
Jun 01 2015 01:43
alright - so I figured it out
Daniel Box
@dbox
Jun 01 2015 01:44
:eyes:
Henry Snopek
@hhsnopek
Jun 01 2015 01:48
so the reason the styles aren't applying is because css_pipeline is concating the bower_component
the js is being concat'ed into the build file tho
Daniel Box
@dbox
Jun 01 2015 01:48
so the js is working properly, just not the css?
Henry Snopek
@hhsnopek
Jun 01 2015 01:48
yeah
I'm looking into the css problem as we speak
Daniel Box
@dbox
Jun 01 2015 01:49
actually
are you sure JS is? (function(){console.log("hello there!")}).call(this); is all i see in build.js
seems like the ignore is just nuking them both
Henry Snopek
@hhsnopek
Jun 01 2015 01:51
yeah - hang on I'll branch and push to gh
Daniel Box
@dbox
Jun 01 2015 01:51
thank you!
alright
so roots cannot ignore bower_components
but
you can manually remove the folder if you'd like after compilation
I've moved the css into a build file so it concats the bower files
Daniel Box
@dbox
Jun 01 2015 01:58
looking
Henry Snopek
@hhsnopek
Jun 01 2015 01:59
if you're using ship to deploy this you could get rid of the after hook in app.coffee then add ignores: ["public/bower_components/**"] to your ship.conf
Daniel Box
@dbox
Jun 01 2015 02:00
so why wouldn't ship have the same problem as before
Henry Snopek
@hhsnopek
Jun 01 2015 02:00
ship is aliased as roots deploy
so ship is separate from roots
ship isn't compiling the files
so it doesn't need to see them
Daniel Box
@dbox
Jun 01 2015 02:01
gotcha
well this is perfect for my needs then I think...
Henry Snopek
@hhsnopek
Jun 01 2015 02:02
that's because we've concat'd all the files into the js/build.js and css/build.css
awesome!
Daniel Box
@dbox
Jun 01 2015 02:02
i imagine other people had similar bower issues
Henry Snopek
@hhsnopek
Jun 01 2015 02:02
yeah
roots v2 had bower intergrated in
Daniel Box
@dbox
Jun 01 2015 02:02
bring it back!
Henry Snopek
@hhsnopek
Jun 01 2015 02:02
haha it would be best to just use an extension or just use the solution we've created here!
Daniel Box
@dbox
Jun 01 2015 02:03
yeah. i'll merge to master and update the readme
so others can reference
i like the manifest
vs having to install more stuff
but rifraf is giong to be dependency right
rimraf
Henry Snopek
@hhsnopek
Jun 01 2015 02:04
that depends
are you going to use ship to deploy?
if you are then you don't need rimraf because when you deploy it will be ignored
Daniel Box
@dbox
Jun 01 2015 02:04
ah
so folder would be there during dev
but not later
Henry Snopek
@hhsnopek
Jun 01 2015 02:05
yeah
Daniel Box
@dbox
Jun 01 2015 02:05
yeah, its funny, I spend all this time to fix this one thing
Henry Snopek
@hhsnopek
Jun 01 2015 02:05
This message was deleted
Daniel Box
@dbox
Jun 01 2015 02:05
that was driving me nuts
but
in the scheme of things it doenst matter
even if it was uploaded
to public
cuz the files aren't referenced
i just didnt like it because it wasn't "neat"
Henry Snopek
@hhsnopek
Jun 01 2015 02:06
haha I know the feeling - that's why I always have that rimraf after hook up my sleeve
you should create an article and submit it to roots-www
Daniel Box
@dbox
Jun 01 2015 02:06
k
yeah, i mean just the repo will be super helpful i think
do you think most people use broweserify
Henry Snopek
@hhsnopek
Jun 01 2015 02:07
exactly and the article will allow everyone to get to it :)
Daniel Box
@dbox
Jun 01 2015 02:07
i just keep reading up on it and seemed like i didn't need it
Henry Snopek
@hhsnopek
Jun 01 2015 02:07
personally I use browserify all the time, but I'll be moving to es6 modules soon
Daniel Box
@dbox
Jun 01 2015 02:09
but for just needing jquery and a few other little things, smashing them all into build.js is fine?
Henry Snopek
@hhsnopek
Jun 01 2015 02:09
well that's dependent on if you really need jquery: http://youmightnotneedjquery.com/
Daniel Box
@dbox
Jun 01 2015 02:10
hehe
i don't even consider myself a developer, so yeah I need it
just a designer who loves to nerd out on stuff
Henry Snopek
@hhsnopek
Jun 01 2015 02:21
haha
merged your stuff to master. hopefully someone else can benefit
thanks again man
Henry Snopek
@hhsnopek
Jun 01 2015 02:27
no problem! make an article out of it :D -https://github.com/carrot/roots-www/tree/master/articles
Daniel Box
@dbox
Jun 01 2015 02:29
the hilarious hting is
i HAD already done the css out
just didn't push it up
Henry Snopek
@hhsnopek
Jun 01 2015 02:29
haha
I would add bower_components to .gitignore
I would add bower_components to .gitignore too
Daniel Box
@dbox
Jun 01 2015 02:30
i'll make the article if you speak to the ship portion
Henry Snopek
@hhsnopek
Jun 01 2015 02:31
haha sure
Daniel Box
@dbox
Jun 01 2015 02:32
I also will wait a bit, because i have several PR's waiting for @jenius
that i want more than this one:)
Unless other people have ability to merge
Henry Snopek
@hhsnopek
Jun 01 2015 02:33
haha I actually have merge approval for the roots-www with the approval from others as well
Daniel Box
@dbox
Jun 01 2015 02:33
haha
k
well you may have to tidy it up a bit. but i'll do my best.
Henry Snopek
@hhsnopek
Jun 01 2015 02:33
sounds good :)
Daniel Box
@dbox
Jun 01 2015 02:33
this is coming from a non developer remember
added bower to gitignore
Henry Snopek
@hhsnopek
Jun 01 2015 02:38
now you just gotta run git rm -r bower_components git push origin master -f
wait
git rm -r --cache bower_components
then push it
Henry Snopek
@hhsnopek
Jun 01 2015 02:42
perfect! :D
Daniel Box
@dbox
Jun 01 2015 02:57
updated readme. let me know if that seems right
Henry Snopek
@hhsnopek
Jun 01 2015 03:15
yeah its good, you could make this into a sprout template if you wanted
it could have a ship.confand everything setup besides the manifests
also if you're using stylus you could always import the files too
Daniel Box
@dbox
Jun 01 2015 03:19
would that allow us to completely get rid of the after?
Henry Snopek
@hhsnopek
Jun 01 2015 03:19
hmm hang on
Daniel Box
@dbox
Jun 01 2015 03:20
i love and use stylus, not sure if everyone does
probably a lot of roots people do
Henry Snopek
@hhsnopek
Jun 01 2015 03:20
everyone doesn't have to use that template ;)
hmm stick with what we have now
Daniel Box
@dbox
Jun 01 2015 03:24
np. works for me
Henry Snopek
@hhsnopek
Jun 01 2015 03:25
its the best way so far
Daniel Box
@dbox
Jun 01 2015 03:25
for some reason i like separating out the bower calls in the manifest, and doing all the stylus imports together
but would be fine with it all in stylus
Henry Snopek
@hhsnopek
Jun 01 2015 03:25
:) there you go
Daniel Box
@dbox
Jun 01 2015 03:27
well this is great. it was the last thing that i needed for roots. thanks again
Henry Snopek
@hhsnopek
Jun 01 2015 03:28
no problem, glad to help!