Comments (7)
Overall, it's freaking AWESOME!
from brunch-guide.
Hey Paul,
- Glou: that's from my original article, and was a nod to the two creators, that I work with now and then. They run the Node.js Paris meetup and mentored at the first NodeSchool Paris. Glou's a pretty good solution, but still entirely code-based (not CoC at all). You're right, it doesn't really bear mention in the "official" guide. I'll strip these.
- Link to Node/npm: yup
- Skeleton standard names: well, the guide de-emphasizes skeletons, and IIRC @es128 was hinting at their being in for a major approach change soon (more generator-like), so this should probably wait?
- Link to docs chapter: I linked to it at the beginning ("this page is your friend"), and I don't think each convention has its own anchor in the docs file, have they?
- Reviewing watcher caveats once 1.8 is out: for sure!
- More info on getting notifications set up: good point, will do.
brunch-server.coffee
: good idea. Won't mention in the docs right now as it isn't the case in 1.7.20, but will update as we go (as for the entire guide, anyway)- Bower install: is there a less cryptic syntax to achieve the same result? Or should I just explain the syntax before or after the CLI?
- Regex preventing autoreload: not sure what the issue is here. What auto-reload? And why won't it get included? From my outsider perspective, my own regexes (my paths + Bower) overflowing to plugin inclusion sounds more like a bug than a regular behavior…?
- Jade compiler slow: oh, that's the reason? You clocked it on my example? I never would have guessed. What about CSS: the LESS preprocessor for my source Bootstrap 3 is fast enough? 👍 for adding a mention of upcoming speedups though.
- Plugin order matters: definitely. Will do.
I'm on a 3-day holiday weekend in Burgundy starting this noon, so I'll be less reactive than usual, but this will get updated and pushed before Sunday evening I guess, so Elan can come in and give it a final glance when he's back 😄
Thanks for the review!
from brunch-guide.
so this should probably wait?
Yes.
Bower install
Let's just describe what the command will do.
Regex preventing autoreload
If the user would install auto-reload-brunch, its vendor file won't get included since it is located in node_modules
, not bower_components
. And the auto-reloading won't work. We don't do this in guide but the user may use the skeleton he made with the guide for future projects.
I never would have guessed
"Compiling" JS files takes approximately zero milliseconds. javascript-brunch just passes the data forward as-is.
Compiling jade, on the other hand, takes considerable amount of time. Parsing, tokenization etc. Just like any other compiler. Some of them are super slow, for example if you use imports in less / stylus — they do synchronous IO. Which is unacceptable to nodejs in my opinion. We have some tricks to speed-up imports in the future though, i'm already improved styl a bit.
Thank you!
from brunch-guide.
Hey @paulmillr, @es128.
Alright, all reviews got taken care of. As both you guys felt 11 was too long, I caved in and split plugin writing in a new chapter, pushing conclusion to 13 (no superstitions around here, I hope 😉)
The demos repo was sync'd, too (regexes in config files, basically).
BTW, about plugin includes get subjected to joinTo
regexes: I consider this counter-intuitive, and borderline buggy. Sure, this creates the issue of "but where do we put these in multiple-targets scenarios?". But this requires end-users to know of plugins' innards, and that kinda sucks, it breaks encapsulation in a big way.
We need to think of a better way to handle this. Perhaps auto-merge in the first file? Or use heuristics as to which ones embed third-party code? Or simply go with the first one and provide new files.*.pluginsJoinTo
settings, that are single strings (target file paths)?
from brunch-guide.
Or use heuristics as to which ones embed third-party code
I think we actually did this in the past. Plugin code had fake paths like vendor/auto-reload.js
. There was some issue with that — I wonder what was it.
This issue would be less relevant when the npm
support would be released. All stuff would be in node_modules
just like brunch plugins.
from brunch-guide.
Uh yeah, I can't double check atm, but I think this concern about the plugin includes was already resolved - and that the regex joinTo would have actually worked fine.
from brunch-guide.
@paulmillr can you browse the latest files with an eye towards this review, and close it if you're happy with the result?
from brunch-guide.
Related Issues (20)
- [TYPO] CONTRIBUTING line 31 - "you house, your rules" -> "your house, your rules" HOT 3
- How to include CSS from npm modules? HOT 4
- Remarques sur le guide français HOT 5
- Apply #20 change to the French guide
- Missing comparison with Webpack HOT 4
- [Chapter 5] JoinTo specific npm packages only HOT 9
- Documentation doesn't mention non-watch incremental build HOT 5
- Dead code elimination HOT 2
- Brunch ?! seems to not work HOT 1
- Add examples of conversion from other build tool(s) HOT 7
- Bower package recognition not working HOT 5
- Jade in Chapter6 HOT 2
- add comments on using ES6 modules with babel HOT 3
- brunch build hangs when brunch-server.js file is present
- Following this guideline, `app.js` isn't created
- Thanx a million ! HOT 1
- Page Not found Error for link to docs page HOT 1
- chapter04-starting-from-scratch.md > Split targets
- Error during node_modules dependency compilation HOT 1
- have to change http://brunch.io/docs/getting-started
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from brunch-guide.