search

RustPython / Parser

MIT License
73 stars 28 forks source link

Add CI for RustPython/Ruff tests #83

Open DimitrisJim opened 1 year ago

DimitrisJim commented 1 year ago

Instead of only running Rust tests, we should also add jobs for running RustPythons (and Ruffs if there's the desire?) tests with every PR.

This will result in a larger wait time but should catch some confusing bugs, i.e https://github.com/RustPython/RustPython/issues/4995

youknowone commented 1 year ago

I'd like to suggest to create tests using this test https://github.com/RustPython/Parser/blob/main/ast-pyo3/test_ast.py We don't need to run RustPython but still can test if the parser result is same as CPython.

DimitrisJim commented 1 year ago

Was thinking more along the lines of a subset, i.e test_grammar.py etc. Anything basically related to parsing would be nice to have.

If it's too much of a hassle, having at least something would definitely be good.