ray/.gitpod at master - hiro/ray - Forgejo: Beyond coding. We Forge.

hiro/ray

mirror of https://github.com/vale981/ray synced 2025-03-04 09:31:43 -05:00

History

Antoni Baum 045c47f172 [CI] Check test files for `if __name__...` snippet (#25322 ) Bazel operates by simply running the python scripts given to it in `py_test`. If the script doesn't invoke pytest on itself in the `if _name__ == "__main__"` snippet, no tests will be ran, and the script will pass. This has led to several tests (indeed, some are fixed in this PR) that, despite having been written, have never ran in CI. This PR adds a lint check to check all `py_test` sources for the presence of `if _name__ == "__main__"` snippet, and will fail CI if there are any detected without it. This system is only enabled for libraries right now (tune, train, air, rllib), but it could be trivially extended to other modules if approved.	2022-06-02 10:30:00 +01:00
..
Dockerfile	[CI] Check test files for `if __name__...` snippet (#25322 )	2022-06-02 10:30:00 +01:00

[CI] Check test files for if __name__... snippet (#25322 )

Bazel operates by simply running the python scripts given to it in `py_test`. If the script doesn't invoke pytest on itself in the `if _name__ == "__main__"` snippet, no tests will be ran, and the script will pass. This has led to several tests (indeed, some are fixed in this PR) that, despite having been written, have never ran in CI. This PR adds a lint check to check all `py_test` sources for the presence of `if _name__ == "__main__"` snippet, and will fail CI if there are any detected without it. This system is only enabled for libraries right now (tune, train, air, rllib), but it could be trivially extended to other modules if approved.

2022-06-02 10:30:00 +01:00

Dockerfile [CI] Check test files for if __name__... snippet (#25322 ) 2022-06-02 10:30:00 +01:00