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

DataizedTest.java:82-85: Fix bug execute an EO program... #623

Closed
0pdd opened this issue May 9, 2022 · 7 comments
Closed

DataizedTest.java:82-85: Fix bug execute an EO program... #623

0pdd opened this issue May 9, 2022 · 7 comments

Comments

@0pdd
Copy link

0pdd commented May 9, 2022

The puzzle 414-aa62591c from #414 has to be resolved:

* @todo #414:30min Fix bug execute an EO program calling a varargs func.
* We reproduced by a test bug during execution with exception `You can't overwrite X`
* when EO app try to call a function that uses varargs as parameter. Now, we must fix it
* and disable the test (test below).

The puzzle was created by @yegor256 on 09-May-22.

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.

@Graur
Copy link
Contributor

Graur commented May 24, 2022

@yegor256 Could you please tell what is the status of this issue?
And is this the same issue as #622?

yegor256 added a commit that referenced this issue May 25, 2022
@0pdd
Copy link
Author

0pdd commented May 25, 2022

The puzzle 414-aa62591c has disappeared from the source code, that's why I closed this issue.

@0pdd 0pdd closed this as completed May 25, 2022
@yegor256
Copy link
Member

@rultor release, tag is 0.23.5

@rultor
Copy link
Contributor

rultor commented May 25, 2022

@rultor release, tag is 0.23.5

@yegor256 OK, I will release it now. Please check the progress here

@rultor
Copy link
Contributor

rultor commented May 25, 2022

@rultor release, tag is 0.23.5

@yegor256 Done! FYI, the full log is here (took me 11min)

@yegor256
Copy link
Member

@Graur should work fine in the latest version

@Graur
Copy link
Contributor

Graur commented May 25, 2022

@yegor256 works great, thanks!

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

No branches or pull requests

4 participants