-
-
Notifications
You must be signed in to change notification settings - Fork 355
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
regressions after big import revamp #3121
Comments
Regression in AssertFixer fixed see STAMP-project/AssertFixer#43 |
Regression in NpeFix fixed see https://ci.inria.fr/sos/job/npefix/971/ |
Regression in Casper fixed, see https://ci.inria.fr/sos/job/casper/1035/ |
regression in gumtree-spoon fixed see https://github.com/SpoonLabs/gumtree-spoon-ast-diff/pull/113/files |
regression in Nopol fixed, see https://ci.inria.fr/sos/job/nopol/618/ |
AssertJ regression fixed, see https://ci.inria.fr/sos/job/AssertJ/1118/ |
next on the list is @seintur's juliac |
Boys, you definitively made some progress! Congrats! One more (last?) NPE to fix in juliac. Visit https://ci.inria.fr/sos/job/juliac/1095/consoleText search for the 2nd occurrence of "Tinfi Example HelloWorld Ultra-Merge" and you will see a stack trace with a NPE at spoon.support.compiler.jdt.ParentExiter.visitCtBlock(ParentExiter.java:442) If this does not speak, I'll try to circumvent the bug to a smaller, more intelligible, case. Tx. |
Juliac is now back to stable https://ci.inria.fr/sos/job/juliac/1102/ (I have commited the needed changes to take into account the revamp). |
Cool, thanks Lionel!
Could you give a link to the commit? I'd like to see the diff and understand the
backward-incompatible change.
|
Hi @monperrus https://gitlab.ow2.org/fractal/juliac/commit/b667b280017a439e75dad164781057ea27d77f84 AFAIK a change in the contract of CTExecutable.toString(). |
Those regressions have to be fixed:
The text was updated successfully, but these errors were encountered: