Skip to content

Support deprecation.rails#89

Merged
yykamei merged 3 commits intomainfrom
deprecation.rails
Apr 4, 2022
Merged

Support deprecation.rails#89
yykamei merged 3 commits intomainfrom
deprecation.rails

Conversation

@yykamei
Copy link
Copy Markdown
Owner

@yykamei yykamei commented Apr 2, 2022

@yykamei yykamei added the enhancement New feature or request label Apr 2, 2022
yykamei added a commit that referenced this pull request Apr 4, 2022
`#children` has been deprecated as of Rails 7.2.

rails/rails#43390

Unlike Rails, I decided to remove `#children` from `BaseEvent`
since it'd be a techdebt to keep this method in the future,
and I guess this method would not be used very well.

By the way, I found this deprecation on Rails when I'm about to
add the support for `deprecation.rails` on #89. I hope this won't cause
problems for users who use this gem.
yykamei added a commit that referenced this pull request Apr 4, 2022
`#children` has been deprecated as of Rails 7.2.

rails/rails#43390

Unlike Rails, I decided to remove `#children` from `BaseEvent`
since it'd be a techdebt to keep this method in the future,
and I guess this method would not be used very well.

By the way, I found this deprecation on Rails when I'm about to
add the support for `deprecation.rails` on #89. I hope this won't cause
problems for users who use this gem.
@yykamei yykamei merged commit efd87b0 into main Apr 4, 2022
@yykamei yykamei deleted the deprecation.rails branch April 4, 2022 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant