Make Rake compile task a dependency of test task

ruby / prism

Prism Ruby parser

https://ruby.github.io/prism/

MIT License

790 stars 134 forks source link

Make Rake compile task a dependency of test task #2887

Closed peterzhu2118 closed 3 weeks ago

peterzhu2118 commented 3 weeks ago

We should make the compile task a dependency of the test task so that:

On new clones of prism, running rake test will not fail with 'Kernel#require_relative': cannot load such file -- prism/lib/prism/node (LoadError).
When a C file changes, it will be recompiled before the tests are ran.

peterzhu2118 commented 3 weeks ago

Ah looks like on CI you want to run tests without compiling prism?

kddnewton commented 3 weeks ago

Yeah we want to be able to install the gem and then test that we can run the tests with the install files to make sure we have all the right stuff bundled.