Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Put @pytest.mark.asyncio decorators back in for now #4598

Closed
wants to merge 2 commits into from

Conversation

MetRonnie
Copy link
Member

Unfortunately, now getting a conflict with pytest-tornasync after removing @pytest.mark.asyncio decorators - eukaryote/pytest-tornasync#13. pytest-tornasync is a dependency of cylc-uiserver, not cylc-flow, so it passed on GH Actions but now flakily fails locally.

This might be fixed by pytest-dev/pytest-asyncio#261 when it makes its way into a release

Requirements check-list

  • I have read CONTRIBUTING.md and added my name as a Code Contributor.
  • Contains logically grouped changes (else tidy your branch by rebase).
  • Does not contain off-topic changes (use other PRs for other changes).
  • Applied any dependency changes to both setup.cfg and conda-environment.yml.
  • Does not need tests
  • No change log entry required
  • No documentation update required.

Getting conflict with pytest-tornasync after removing `@pytest.mark.asyncio` decorators - eukaryote/pytest-tornasync#13

Partial revert of 7b4e95c, reversing
changes made to eb2e4fd.
@MetRonnie MetRonnie self-assigned this Jan 20, 2022
@MetRonnie MetRonnie changed the title Partial revert of #4596 Put @pytest.mark.asyncio decorators back in for now Jan 20, 2022
@MetRonnie MetRonnie added the infrastructure GH Actions, Codecov etc. label Jan 20, 2022
@oliver-sanders
Copy link
Member

I think I've got an alternative that disables the pytest_tornasync plugin to prevent it from interfering so we can keep your earlier changes.

@MetRonnie MetRonnie marked this pull request as draft January 20, 2022 13:59
@MetRonnie
Copy link
Member Author

Superseded by #4599

@MetRonnie MetRonnie closed this Jan 20, 2022
@MetRonnie MetRonnie deleted the patch branch January 20, 2022 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants