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
Doc fixes, minor examples cleanup #2198
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
my thoughts
@@ -1,5 +1,4 @@ | |||
var fs = require('fs'); | |||
var url = require('url'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is just an unused import.
|
||
server.listen(PORT); | ||
|
||
console.log("Examples running on " + PORT); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Had to read the source code to figure this out. Adding a console log for future devs
* simplify language
bootstrap 4 is slightly smaller then 3.2.0 so it should also help load examples faster
surface user/consumer changes first
@felipewmartins rebased out the README fixes already merged can you have a look? it's focused on general local dev stuff, no source changes |
Hi @avindra |
* README.md COOKBOOK.md: minor fixes * simplify language * ECOSYSTEM: create a few categories * Examples: log port listening to * upgrade bootstrap 3 -> 4 in examples bootstrap 4 is slightly smaller then 3.2.0 so it should also help load examples faster * categorize 0.19 items a little differently surface user/consumer changes first
validateStatus
andeject
for the interceptorMinor: update to bootstrap 4 build in examples files