ray icon indicating copy to clipboard operation
ray copied to clipboard

[core] Remove tests that are permanently skipped with old decorator

Open edoakes opened this issue 7 months ago • 4 comments

This decorator marks the tests as always skipped (we don't set the env variable anywhere).

The tests have been disabled for a long time, so just remove them.

edoakes avatar May 15 '25 14:05 edoakes

@jjyao any of these you think are important enough that warrants fixing them instead of deleting?

I'm actually a bit confused because we seem to have "fixed" some, like this: https://github.com/ray-project/ray/issues/41175

But we never actually re-enabled the test?

edoakes avatar May 15 '25 14:05 edoakes

going to go through and be a bit more careful about which we remove/dedup/enable

edoakes avatar May 19 '25 16:05 edoakes

This pull request has been automatically marked as stale because it has not had any activity for 14 days. It will be closed in another 14 days if no further activity occurs. Thank you for your contributions.

You can always ask for help on our discussion forum or Ray's public slack channel.

If you'd like to keep this open, just leave any comment, and the stale label will be removed.

github-actions[bot] avatar Jun 15 '25 00:06 github-actions[bot]

asdf

edoakes avatar Jun 16 '25 14:06 edoakes

This pull request has been automatically marked as stale because it has not had any activity for 14 days. It will be closed in another 14 days if no further activity occurs. Thank you for your contributions.

You can always ask for help on our discussion forum or Ray's public slack channel.

If you'd like to keep this open, just leave any comment, and the stale label will be removed.

github-actions[bot] avatar Jul 01 '25 00:07 github-actions[bot]