eslint-rails icon indicating copy to clipboard operation
eslint-rails copied to clipboard

Fix Rails 5.1; use before_action instead of before_filter

Open scottbartell opened this issue 7 years ago • 8 comments

before_filter is deprecated and was removed in Rails 5.1.

Note: I believe that before_action is supported in Rails 4 but not in Rails 3. IMO, if this gem does currently work in Rails 3 (I have no idea if it does), dropping support might not be too bad of an idea given that Rails 3.x has been officially unsupported since June 2016.

scottbartell avatar Aug 31 '17 21:08 scottbartell

Plus 💯

I am getting the following error because of this...

rake aborted!
NoMethodError: undefined method `before_filter' for EslintController:Class
Did you mean?  before_action
(See full trace by running task with --trace)```

chrishough avatar Oct 24 '17 06:10 chrishough

Hey all, not sure what the timeline is for this repo, but I have forked it and included this PR on the repo located here: https://github.com/DLvalentine/eslint-rails-ee

Cheers

DLvalentine avatar Feb 19 '18 23:02 DLvalentine

Hey @jonkessler let's get this PR merged. We are getting error in Rails version upgrade.

aadill77 avatar Mar 23 '18 15:03 aadill77

@aadill77, first off, I don’t work for the company anymore so I can’t merge this. Second off, this is open source. You’re getting this work for free and you don’t get to demand that people do work for you.

Maybe @amutz can help get this merged.

jonkessler avatar Mar 23 '18 15:03 jonkessler

FWIW @aadill77 , I have a fork here https://github.com/dlvalentine/eslint-rails-ee that did merge in this PR

DLvalentine avatar Mar 23 '18 15:03 DLvalentine

@jonkessler sorry for that rude tone. Didn't realize it until I read it again myself. My bad!

aadill77 avatar Mar 23 '18 15:03 aadill77

@DLvalentine Thanks for your help.

aadill77 avatar Mar 23 '18 15:03 aadill77

I'm seeing this as well, any timeline on when this will be merged?

JerrodCarpenter avatar Apr 11 '18 16:04 JerrodCarpenter