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

Fix fork io testing #126

Merged
merged 4 commits into from
Nov 12, 2024
Merged

Conversation

hhefesto
Copy link
Contributor

@hhefesto hhefesto commented Nov 8, 2024

Fixes cabal test

Now telomare runs can a priori be given a list of inputs that'll be fed
when input is required. Also the output of telomare runs can now be
captured which is nice for testing.

Now telomare runs can a priori be given a list of inputs that'll be fed
when input is required. Also the output of telomare runs can now be
captured which is nice for testing.

Also some cleaning.
@hhefesto hhefesto force-pushed the fix-fork-IO-testing branch from c4e6eff to c7f73af Compare November 8, 2024 02:13
Copy link
Collaborator

@sfultong sfultong left a comment

Choose a reason for hiding this comment

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

remove debugging, clean up newline issue if you want

I've been thinking: do you want me to give you more stylistic feedback? Sometimes I think I could do things in a cleaner way, but it's fine as-is

src/Telomare/Resolver.hs Show resolved Hide resolved
src/Telomare/Eval.hs Outdated Show resolved Hide resolved
@hhefesto
Copy link
Contributor Author

I've been thinking: do you want me to give you more stylistic feedback? Sometimes I think I could do things in a cleaner way, but it's fine as-is

any input is more than welcomed : )

Also inverted the arguments of `@?=` where necessary (right side is for
expected value) and did some cleanup
@sfultong sfultong merged commit f3cf568 into Stand-In-Language:master Nov 12, 2024
4 checks passed
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