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.

Sign up for GitHub

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

Jump to bottom

Data.Text.Foreign: added peekCString #599

Merged
merged 2 commits into from
Jul 22, 2024
Merged

Data.Text.Foreign: added peekCString #599

merged 2 commits into from
Jul 22, 2024

Conversation

TristanCacqueray
Copy link
Contributor

Fixes: #239

@Bodigrim
Copy link
Contributor

@TristanCacqueray could you add a test please?

@TristanCacqueray
Copy link
Contributor Author

@Bodigrim sure, well there was no test for Data.Text.Foreign, so I added new ioProperties. Please let me know if that's what you want.

@Bodigrim Bodigrim requested a review from Lysxia June 3, 2024 22:25
@TristanCacqueray
Copy link
Contributor Author

@Lysxia Hello, could you please have a look?

@Lysxia
Copy link
Contributor

Lysxia commented Jul 22, 2024

This fell off my todolist. Sorry for the delay!

@Lysxia Lysxia merged commit 1063d61 into haskell:master Jul 22, 2024
28 checks passed
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Bodigrim Bodigrim Bodigrim approved these changes

@Lysxia Lysxia Lysxia approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

add Data.Text.Foreign.peekCString?
3 participants
@TristanCacqueray @Bodigrim @Lysxia