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

io: merge eval_ast into eval, add DEBUG-EVAL #699

Merged
merged 4 commits into from
Oct 29, 2024

Conversation

asarhaddon
Copy link
Contributor

The self-hosted tests fail, for apparently unrelated reasons (the first readline fails in step0). Were they failing before?

@kanaka
Copy link
Owner

kanaka commented Oct 16, 2024

@asarhaddon @dubek The update to the version of io in the Dockerfile appears to have completed self-hosted tests successfully: https://github.com/kanaka/mal/actions/runs/11327008274/job/31497159120 The io repo hasn't changed from the time when @dubek did the update to when the pushed image would have been pushed (although it would have been a different instance of the build but of the same sources).

Maybe it's some sort of temperamental GC related issue? When I build and run locally with it works.

@asarhaddon
Copy link
Contributor Author

Sorry, I have no clue and miss motivation to investigate this issue.

@asarhaddon
Copy link
Contributor Author

A single comma was causing a crash reported in an unrelated module...
And now that this is solved, a self-hosting issue.
I have no clue for now, but I suggest to move this enigm from #699 to #662.

@kanaka kanaka merged commit dfc67ee into kanaka:master Oct 29, 2024
3 of 4 checks passed
@kanaka
Copy link
Owner

kanaka commented Oct 29, 2024

@asarhaddon sounds like a reasonable plan. I've added io to #662

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