Jared Beck

Results 80 comments of Jared Beck

In my experience, when a `Net::ReadTimeout` occurs, then `HTTParty.post` will return an actual `nil`. If I'm correct (it's not documented, just my observation) then this is something everyone should check...

@dohpaz42 thanks, worked for me. Make sure to use https://github.com/andialbrecht/sqlparse/tree/master/sqlparse not https://github.com/andialbrecht/sqlparse

https://github.com/paper-trail-gem/paper_trail/pull/1108 didn't work out. Instead, we'll add an optional column `item_subtype` to our `versions` table(s), per https://github.com/paper-trail-gem/paper_trail/pull/1143 The addition of this column will be mentioned under the "Fixed" section of...

> @blackwinter suggested to try ignoring warnings by :ignorewarnings => true in MediaWiki::Gateway.new - I confirm that it works around the issue. Is it still necessary to use this workaround?

Here's another warning from MW 1.25, raised by `MediaWiki::Gateway::Users#options`. > MediaWiki::APIError: API error: code 'warning', info 'API warning: action=tokens has been deprecated. Please use action=query&meta=tokens instead.'

> .. given that WMF has now released an "official" client .. I will likely be deprecating this library entirely .. Makes sense, thanks.

> Perhaps this can be added to the `LineLength` cop as some option, as it already breaks down long arrays. I just don't recall how exactly did it split them....

@bbatsov have you reached a decision about which cop this belongs in? Thanks.

> .. perhaps there could be a configuration option in which users can define exactly which operators they'd like included/excluded, with all included by default? :+1: but with arithmetic operators...

> Yes, the problem I reported here is that `git pr checkout ` did not add the source remote used in the PR. To add some context: Currently, to push...