Roman Shtylman

Results 321 comments of Roman Shtylman

less .json

+1 Agreed. My longterm hope is that a package manager understands history.md or something and just grabs the version from that on publish.

less .json

If there is no version field in a json file then should it be ignored? (There might be json files that are unrelated to the package). I am skeptical about...

less .json

which files to whitelist then? package.json component.json any others?

Add index.d.ts with Typescript types

Any desire to merge this?

Add index.d.ts with Typescript types

@qiao ?

Add browser build step for minified browser version using querySelectorAll polyfill

use `make dist` to try it out

Please add app.route('foo').use()

We decided against this because .use carries a specific meaning with what it does to req.path

Please add app.route('foo').use()

I need to check on the OPTIONS behavior but I would lean towards making sure `.all` doesn't affect OPTIONS for `.route` or `app.all()` but that might break some other assumptions...

Please add app.route('foo').use()

I think we should change `.all` to simply add middleware and not affect options responses. Yes, it will be slightly different than app.all, but whatever.

Please add app.route('foo').use()

Sounds like a 5.x feature maybe. Along with possibly ripping out the routing system into a module finally.