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 the explode test case added in #231 #232

Merged
merged 1 commit into from
Nov 22, 2022

Conversation

itchyny
Copy link
Contributor

@itchyny itchyny commented Nov 21, 2022

I'm sorry 😢, I failed to run the test on my laptop and expected to be run on CI, but it somehow passes without running the test?

@eiiches
Copy link
Owner

eiiches commented Nov 22, 2022

I didn't notice it either, sorry about that. I'll have to look if the test is working properly on CI later.

@eiiches eiiches merged commit fcbe2bb into eiiches:develop/1.x 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.

2 participants