Skip to content

CI: Use xsel instead of xclip for numpydev #45576

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

Merged
merged 5 commits into from
Jan 24, 2022
Merged

Conversation

mroeschke
Copy link
Member

Hopefully fixes this flaky clipboard test

_________________________ test_raw_roundtrip[abcd...] __________________________
[gw1] linux -- Python 3.10.0 /usr/share/miniconda/envs/pandas-dev/bin/python

data = 'abcd...'

    @pytest.mark.single
    @pytest.mark.clipboard
    @pytest.mark.parametrize("data", ["\U0001f44d...", "Ωœ∑´...", "abcd..."])
    def test_raw_roundtrip(data):
        # PR #25040 wide unicode wasn't copied correctly on PY3 on windows
        clipboard_set(data)
>       assert data == clipboard_get()
E       AssertionError: assert 'abcd...' == 'Ωœ∑´...'
E         - Ωœ∑´...
E         + abcd...

pandas/tests/io/test_clipboard.py:313: AssertionError

@mroeschke mroeschke added the CI Continuous Integration label Jan 23, 2022
@mroeschke mroeschke added this to the 1.5 milestone Jan 24, 2022
@mroeschke mroeschke modified the milestones: 1.5, 1.4.1 Jan 24, 2022
@jreback jreback merged commit ea2b0fd into pandas-dev:main Jan 24, 2022
@jreback
Copy link
Contributor

jreback commented Jan 24, 2022

green!! yay

@jreback
Copy link
Contributor

jreback commented Jan 24, 2022

@meeseeksdev backport 1.4.x

@lumberbot-app
Copy link

lumberbot-app bot commented Jan 24, 2022

Something went wrong ... Please have a look at my logs.

@mroeschke mroeschke deleted the ci/xsel branch January 24, 2022 22:51
jreback pushed a commit that referenced this pull request Jan 25, 2022
yehoshuadimarsky pushed a commit to yehoshuadimarsky/pandas that referenced this pull request Jul 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants