rubocop / rubocop-ast

RuboCop's AST extensions and NodePattern functionality
https://docs.rubocop.org/rubocop-ast
MIT License
104 stars 52 forks source link

Remove prism as a runtime dependency #283

Closed marcandre closed 7 months ago

marcandre commented 7 months ago

Fixes #282

koic commented 7 months ago

It is mentioned below, it seems better to hold off on making a decision to remove it for now: https://github.com/rubocop/rubocop-ast/issues/282#issuecomment-1972341899

If the decision is made to remove it, it would likely require adjustments not only to the removing dependency, also to issues surrounding Prism's loading and documentation. It might be better if I handle it. Thank you.

koic commented 7 months ago

I've opened #285. It addresses users who don't have Prism installed, indicating through warning and documentation that Prism needs to be installed separately.

bbatsov commented 7 months ago

Addressed by #285.