-
Notifications
You must be signed in to change notification settings - Fork 650
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
The test cases cannot have quotes (") #60
Comments
Hi @Licheng-Xu 可以提供一下题号吗? |
不好意思,这几天忙着其他事情了。 题号:[13] 选择直接输入测试样例,输入 "LVIII",之后返回了
|
@Licheng-Xu It looks like a bug. Thank you for the feedback. 👍 |
Hi @jdneo , looks like the bug happened because in test.ts: when we get the test string, we replace all /" with "", which drops quotes. Thus in leetcode, it can't recognize the input as a string. |
@LiuLiC Yeah, I know that. Because I remember that there is a problem if we do not do the replacement. Not quite sure how the current status is. PR is welcome if you are willing to fix this. 😄 |
Fixed in #111. Will get released in |
Hmm... Actually, this bug is not well fixed in #111. I did some experiments on different platforms and found that the test string needs to be parsed in different ways: Bash (Mac)
CMD (Windows)
PowerShell (Windows)
I have no Linux machine to verify, but guess the behavior should be similar with that in So I created another PR: #113, to fix it. |
@ALL 0.10.2 has been released with this fix. |
尝试使用 "00000" 或者 "00000" 都不行
The text was updated successfully, but these errors were encountered: