Skip to content

Cleaned up GOOLTest File#4722

Merged
JacquesCarette merged 8 commits intomainfrom
cleanup-main
Feb 14, 2026
Merged

Cleaned up GOOLTest File#4722
JacquesCarette merged 8 commits intomainfrom
cleanup-main

Conversation

@B-rando1
Copy link
Collaborator

Builds on #4721 (was necessary for some of the functions I used). Might be good to review that one first.

There were some similar improvements in that PR that I wanted to apply to Main.hs, so I did them here. Thanks to @TOTBWF for helping me untangle genCode!

We also enabled a bunch of language extensions along the way 😅 hopefully those are good changes as well.

Copy link
Owner

@JacquesCarette JacquesCarette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Modulo changes needed in #4721 , the changes here are good.

genCode files = createCodeFiles (concatMap (\p -> replicate (length (progMods
(packageProg p)) + length (packageAux p)) (progName $ packageProg p)) files) $
makeCode (map (progMods . packageProg) files) (map packageAux files)
genCode files =
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

much nicer!

@JacquesCarette JacquesCarette merged commit 092f539 into main Feb 14, 2026
5 checks passed
@JacquesCarette JacquesCarette deleted the cleanup-main branch February 14, 2026 20:49
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.

3 participants