Marshall Cottrell

LGTM

See [the autoresize demo](http://ajaxorg.github.io/ace/demo/autoresize.html). Basically you'll want this in your `onLoad` callback: ``` js editor.setOptions({ autoScrollEditorIntoView: true, maxLines: 50 }) ``` Note that you should avoid setting `maxLines` to `Infinity`...

This may be a more general VSCode feature request since this plugin appears to defer to the built-in diff previewing functionality, but I wasn't totally sure.

@bgrant0607 @ash2k I think it might be a mistake to conflate the discussion around how "inventory metadata" is persisted to the cluster (which may very well be tooling specific) and...

@19majkel94 related to your suggested tweaks to my proposal in https://github.com/pleerock/routing-controllers/issues/144#issuecomment-321973571, I think we could go much farther in simplifying things by ditching entirely the concept of "middleware" as it...

@NoNameProvided to answer your questions: > @marshall007 When the body is parsed in the lifecycle? Before the `before` hook or after? Do you mean the request body? I was imagining...

@NoNameProvided to better answer your last question, I think we should strive to make `Interceptors` eliminate the need for a `next()` callback just like controller methods do.

> With this you force people to wrap classic express middlewares into interceptors classes or again do awful configuration: ... @19majkel94 I don't see the "awful configuration" example you mentioned...

@19majkel94 I think you'd want Interceptor 1 to implement `before` and `intercept`. Then your order would be [2, 1].

We could disallow modifying the response in `AfterInterceptor` to make it clear that formatting is supposed to be handled in the `intercept` method.