Chris Bottin

Results 8 comments of Chris Bottin

btw, happy to create a PR for it if the changes are sound.

Thanks for your message. According to you, what should be the rule to wrap the attributes into the next line ?

I will look into it if that can be done easily but I can't give you a time estimate.

Thanks for your comment @dgergely, I agree it's a good approach. I might take a look this week.

Hey @MaxInMoon I think the behaviour is correct the root `span` element contains a text node `{"type":"Text","content":" "}` therefore the `collapseContent=true` option behaves correctly, all the content inside the element...

I'm afraid we might upset some people if we do that. But there could be a compromise by using https://nodejs.org/api/os.html#os_os_eol to detect what should be used if the `lineSeparator` option...

@ugpeter `authHeader` doesn't yet accept the `extra_params` parameter. I suggest you change your code to use `oauth._prepareParameters(token, secret, method, url, params)` instead of `oauth.authHeader(url, token, secret, method)`. This is what...

Thanks @smohanty92 for submitting your issue. I will see if there is anything obvious causing this memory issue.