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

BinarizeMojoTest.java:48-50:... #1986

Closed
0pdd opened this issue Apr 12, 2023 · 5 comments · Fixed by #1988
Closed

BinarizeMojoTest.java:48-50:... #1986

0pdd opened this issue Apr 12, 2023 · 5 comments · Fixed by #1988
Assignees

Comments

@0pdd
Copy link

0pdd commented Apr 12, 2023

The puzzle 1829-71075212 from #1829 has to be resolved:

* @todo #1829:30min BinarizeMojoTest.binarizesWithoutErrors() fails sometimes.
* Apparently, the test was merged without appropriate checks verification.
* The test should be fixed and then enabled.

The puzzle was created by volodya-lombrozo on 11-Apr-23.

Estimate: 30 minutes, role: DEV.

If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@volodya-lombrozo
Copy link
Member

@levBagryansky Could you have a look, please?

@levBagryansky
Copy link
Member

@volodya-lombrozo sure. Maybe you still have a stack trace error?

@volodya-lombrozo
Copy link
Member

@levBagryansky We have already fix it.

levBagryansky added a commit to levBagryansky/eo that referenced this issue Apr 12, 2023
levBagryansky added a commit to levBagryansky/eo that referenced this issue Apr 12, 2023
levBagryansky added a commit to levBagryansky/eo that referenced this issue Apr 12, 2023
levBagryansky added a commit to levBagryansky/eo that referenced this issue Apr 12, 2023
levBagryansky added a commit to levBagryansky/eo that referenced this issue Apr 12, 2023
@levBagryansky
Copy link
Member

levBagryansky commented Apr 12, 2023

@volodya-lombrozo I run mvn clean install locally many times and restarted checks in pr #1988. There was not error you reported. Please provide more information about the bug.

@volodya-lombrozo
Copy link
Member

@levBagryansky Let's just remove the puzzle and close the issue. Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants