Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hotfix: Fix deprecated method warning. #1315

Closed

Conversation

lanzhiheng
Copy link
Contributor

@lanzhiheng lanzhiheng commented Jul 27, 2020

fixes: #1314

@mauro-oto
Copy link

@justinfrench any chance you could review this PR? Thank you!

@mauro-oto
Copy link

Ping @mikz, would you have some time to review this fix? Thank you! 🙏

@tute
Copy link

tute commented Aug 25, 2020

There's a new duplicate here: #1317

@mikz
Copy link
Contributor

mikz commented Aug 26, 2020

Hey. I prefer #1317 as it is in the spirit of the original code.
But as mentioned there, I believe this code path is not triggered in the test suite as I can't see the deprecation warning in the test log.
I think it needs a test case to trigger the behaviour and then fix it to make sure we are not breaking anything.

@deivid-rodriguez
Copy link
Member

Hi! Can you try 4.0.0.rc1 and see if this issue is fixed there?

@tute
Copy link

tute commented Dec 1, 2020

I can confirm there are warnings with 3.1.5 but not with 4.0.0.rc1. I also have tests failing but I'm not sure whether they are directly related with the upgrade, and I believe that's independent. Thanks!

@deivid-rodriguez
Copy link
Member

Thanks for confirming @tute, I'll close this, then.

@deivid-rodriguez
Copy link
Member

Actually I don't have permissions. @mikz can you close this and the other PR?

@mikz
Copy link
Contributor

mikz commented Dec 1, 2020

Does someone have the commit/PR that fixed it?

Closing. Fixed in https://github.com/formtastic/formtastic/releases/tag/4.0.0.rc1

edit: I can't close the issue either 🤡

@deivid-rodriguez
Copy link
Member

I don't... It'd be a matter of running a git bisect, I guess.

@justinfrench
Copy link
Member

@mikz @deivid-rodriguez I think I've fixed your permissions dos that you can close issues (among many other things) ;)

@deivid-rodriguez
Copy link
Member

Thanks so much @justinfrench! Closing this then :)

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.

Warning for deprecated method Object#=~
6 participants