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.

Sign up for GitHub

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

Jump to bottom

Fix deprecations and other warnings in tests #333

Merged
merged 7 commits into from
Jul 2, 2023
Merged

Fix deprecations and other warnings in tests #333

merged 7 commits into from
Jul 2, 2023

Commits on Jun 28, 2023

  1. Fix deprecated threading methods

    @RazerM
    RazerM committed Jun 28, 2023
    Configuration menu
    Copy the full SHA
    9d51f16 View commit details
    Browse the repository at this point in the history

  2. Remove some remaining Python 2 stuff

    @RazerM
    RazerM committed Jun 28, 2023
    Configuration menu
    Copy the full SHA
    5ae321e View commit details
    Browse the repository at this point in the history

  3. Fix __del__ when ZeroMQHandler is partially initialised

    @RazerM
    RazerM committed Jun 28, 2023
    Configuration menu
    Copy the full SHA
    d674615 View commit details
    Browse the repository at this point in the history

  4. Add ids to parameterized fixtures

    @RazerM
    RazerM committed Jun 28, 2023
    Configuration menu
    Copy the full SHA
    6fdd5f2 View commit details
    Browse the repository at this point in the history

  5. Test gevent in tox (and therefore CI)

    @RazerM
    RazerM committed Jun 28, 2023
    Configuration menu
    Copy the full SHA
    fb66a5f View commit details
    Browse the repository at this point in the history

Commits on Jul 2, 2023

  1. Fix deprecated/unused imports

    @RazerM
    RazerM committed Jul 2, 2023
    Configuration menu
    Copy the full SHA
    f37cb43 View commit details
    Browse the repository at this point in the history

  2. Prefer contextvars ident over greenlet ident 

    Without this, the following test fails:

tests/test_asyncio.py::test_asyncio_context_management[gevent]

I don't know how realistic such a scenario is, but it seems logical
that if we prefer context_get_ident over thread_get_ident then the
same should apply to greenlet_get_ident.
    @RazerM
    RazerM committed Jul 2, 2023
    Configuration menu
    Copy the full SHA
    ccfc4cf View commit details
    Browse the repository at this point in the history