Skip to content

Don't ary.inspect in the lint assertions#1626

Merged
ioquatix merged 2 commits intorack:masterfrom
ziggythehamster:fix-1625
Mar 24, 2020
Merged

Don't ary.inspect in the lint assertions#1626
ioquatix merged 2 commits intorack:masterfrom
ziggythehamster:fix-1625

Conversation

@ziggythehamster
Copy link
Contributor

Fixes #1625

@ziggythehamster
Copy link
Contributor Author

I'm not sure what the deal is with circleci

@ioquatix ioquatix merged commit 026537f into rack:master Mar 24, 2020
@bpinto
Copy link

bpinto commented Jun 25, 2020

I've notice a new release for 2.x is out but it does not include this patch. Would it be possible to back port this commit into 2.x branch?

@ioquatix
Copy link
Member

There is no reason why it couldn't be back ported. I would say that this was a bug fix more than a feature/enhancement so it is a valid candidate for a 2.2.x update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2.2.x Rack::Lint severe performance regression

3 participants