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

fix: empty session name #1959

Merged
merged 1 commit into from
Nov 22, 2022
Merged

fix: empty session name #1959

merged 1 commit into from
Nov 22, 2022

Conversation

jaeheonji
Copy link
Member

fix #1939

@jaeheonji
Copy link
Member Author

@imsnif Hey, sorry for the ping 😅 I'd like some advice.

Strangely, CI is getting an error about the tests::e2e::cases::send_command_through_the_cli test case.
There was no problem when testing locally. And I think the change point of this PR is irrelevant to the test case.

Can I ignore this?

@jaeheonji jaeheonji temporarily deployed to cachix November 22, 2022 08:58 Inactive
@imsnif
Copy link
Member

imsnif commented Nov 22, 2022

Hey @jaeheonji - sorry about the flaky e2e test. I ran it again and it passed. It happens sometimes (I keep making them more robust, but GH actions runs them on just 1 CPU, so there's only so much we can do...)

If after a re-run it passes, then free free to merge.

@jaeheonji
Copy link
Member Author

@imsnif Thank you for your kindness.

@jaeheonji jaeheonji merged commit 211d3b1 into main Nov 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

zellij with empty session name will be blocked forever
2 participants