Skip to content

Replace deprecated __find_args__ with Array#flatten - #279

Merged
dblock merged 2 commits into
mongoid:masterfrom
aurelien-reeves-scalingo:fix/remove_deprecated_find_args
May 22, 2026
Merged

dblock merged 2 commits into
mongoid:masterfrom
aurelien-reeves-scalingo:fix/remove_deprecated_find_args

Conversation

@aurelien-reeves-scalingo

Copy link
Copy Markdown
Contributor

__find_args__ is deprecated in Mongoid 9.x and will be removed in Mongoid 10.0 It's functionally equivalent to Array#flatten
This fixes the deprecation warning: DEPRECATION WARNING: find_args is deprecated and will be removed from Mongoid 10.0

`__find_args__` is deprecated in Mongoid 9.x and will be removed in Mongoid 10.0
It's functionally equivalent to `Array#flatten`
This fixes the deprecation warning: DEPRECATION WARNING: find_args is deprecated and will be removed from Mongoid 10.0

Co-authored-by: samiFakhfakhScalingo <sami.fakhfakh@scalingo.com>
@github-actions

github-actions Bot commented May 22, 2026 •

Copy link
Copy Markdown

Danger Report

No issues found.

View run

@aurelien-reeves-scalingo
aurelien-reeves-scalingo force-pushed the fix/remove_deprecated_find_args branch from 7154634 to 3916ee1 Compare May 22, 2026 13:25
@dblock
dblock merged commit eb79653 into mongoid:master May 22, 2026
14 checks passed
@dblock

dblock commented May 22, 2026

Copy link
Copy Markdown
Collaborator

@aurelien-reeves-scalingo take a look at #275, anything we need from there?

@aurelien-reeves-scalingo

aurelien-reeves-scalingo commented May 22, 2026 •

Copy link
Copy Markdown
Contributor Author

Indeed, multi_arged? is deprecated too
Could be nice to get rid of it

But I don't know if changes from #275 are relevant as they broke the CI.

The author mentioned that the failures were "unrelated to its PR". To begin with, trying to re-run the CI on #275 to see if more recent versions of Mongoid have fixed the failures may be a first step. WDYT?

@dblock

dblock commented May 22, 2026

Copy link
Copy Markdown
Collaborator

@aurelien-reeves-scalingo just extract/rebase this PR into a new one?

@aurelien-reeves-scalingo

Copy link
Copy Markdown
Contributor Author

Yes, we can give it a try

@aurelien-reeves-scalingo

Copy link
Copy Markdown
Contributor Author

Here it is: #280

@aurelien-reeves-scalingo
aurelien-reeves-scalingo deleted the fix/remove_deprecated_find_args branch May 27, 2026 12:56
@aurelien-reeves-scalingo

Copy link
Copy Markdown
Contributor Author

@dblock is it planned to release recent changes to the gem?

@dblock

dblock commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

I released 7.1.0. Sorry for the delay.

@aurelien-reeves-scalingo

Copy link
Copy Markdown
Contributor Author

Great, thanks a lot!

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 participants