search

whitequark / parser

A Ruby parser.
Other
1.57k stars 197 forks source link

Use `AST::Processor::Mixin` instead of deprecated `AST::Processor` #1000

Closed koic closed 4 months ago

koic commented 4 months ago

This PR uses AST::Processor::Mixin instead of deprecated AST::Processor API:

# This class includes {AST::Processor::Mixin}; however, it is
# deprecated, since the module defines all of the behaviors that
# the processor includes.  Any new libraries should use
# {AST::Processor::Mixin} instead of subclassing this.
#
# @deprecated Use {AST::Processor::Mixin} instead.

https://github.com/whitequark/ast/blob/v2.4.2/lib/ast/processor.rb#L2-L7

There is an imcompatibility where the inheritance no longer includes AST::Processor, but the impact is expected to be negligible.

iliabylich commented 4 months ago

Makes sense, thanks!