Dead code from ast

[adbatista]

I cleaned The dead code from AST, the the tests in cucumber and cucumber-core are OK

[mattwynne]

Do these still both need to be mutable? Or would attr_reader suffice?

[mattwynne]

@adbatista thanks for this, and sorry it's taken so long to give it full attention - bad time of year!

@tooky this LGTM, any comments?

[adbatista]

No problem
I made the rabase from master branch and changed the attr_accessor :exception, :multiline_arg to attr_reader
the cucumber tests are all green

[tooky]

This looks fine from my POV.

thanks @adbatista.

Perhaps we should make sure the cucumber/cucumber tests still pass with this branch?

[adbatista]

I executed the specs and features on cucumber/cucumber
they are all green

[mattwynne]

Thanks for your contribution @adbatista! I look forward to the next one 😄

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.