Skip to content

Backport-7030: Fix uninitialized constant Parser::AST::Processor::Mixin (#7030)#7036

Merged
ahasunos merged 1 commit intoinspec-5from
ss/bp-7030
May 2, 2024
Merged

Backport-7030: Fix uninitialized constant Parser::AST::Processor::Mixin (#7030)#7036
ahasunos merged 1 commit intoinspec-5from
ss/bp-7030

Conversation

@ahasunos
Copy link
Contributor

@ahasunos ahasunos commented May 2, 2024

Description

Backports #7030

Related Issue

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New content (non-breaking change)
  • Breaking change (a content change which would break existing functionality or processes)

Checklist:

  • I have read the CONTRIBUTING document.

parser 3.3.1.0 introduced
whitequark/parser#1000, which causes this
failure:

```
% bundle exec ruby -Itest test/unit/profiles/profile_test.rb
inspec/lib/inspec/utils/profile_ast_helpers.rb:7:in `<class:CollectorBase>': uninitialized constant Parser::AST::Processor::Mixin (NameError)

        include Parser::AST::Processor::Mixin
                                      ^^^^^^^
```

Fix this by inherting from `Parser::AST::Processor` instead.

Closes #7029

Signed-off-by: Stan Hu <stanhu@gmail.com>
@ahasunos ahasunos requested a review from a team as a code owner May 2, 2024 08:53
@ahasunos ahasunos requested review from clintoncwolfe and sathish-progress and removed request for a team May 2, 2024 08:53
@ahasunos ahasunos merged commit 0a0acec into inspec-5 May 2, 2024
@ahasunos ahasunos deleted the ss/bp-7030 branch May 2, 2024 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants