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

Move sameError to fs-sim-test #64

Merged
merged 1 commit into from
May 31, 2024
Merged

Move sameError to fs-sim-test #64

merged 1 commit into from
May 31, 2024

Conversation

jorisdral
Copy link
Collaborator

@jorisdral jorisdral commented May 23, 2024

The sameError function is only used by the model-based tests in fs-sim-test to compare errors that the mocked file system throws against errors that the real file system throws. We do not expect users of the fs-api/fs-sim packages to use this comparison function themselves, so we have moved to fs-sim-test instead.

Modules in the System.FS.IO.Internal hierarchy are now also public

@jorisdral jorisdral self-assigned this May 23, 2024
@jorisdral jorisdral marked this pull request as ready for review May 23, 2024 15:18
@jorisdral jorisdral requested a review from a team as a code owner May 23, 2024 15:18
fs-api/fs-api.cabal Outdated Show resolved Hide resolved
@jorisdral jorisdral force-pushed the jdral/move-same-error branch 5 times, most recently from 5e9a674 to 77283a4 Compare May 31, 2024 16:26
The `sameError` function is only used by the model-based tests in `fs-sim-test`
to compare errors that the mocked file system throws against errors that the
real file system throws. We do not expect users of the `fs-api`/`fs-sim`
packages to use this comparison function themselves, so we have moved to
`fs-sim-test` instead.

Modules in the `System.FS.IO.Internal` hierarchy are now also public.
@jorisdral jorisdral added this pull request to the merge queue May 31, 2024
Merged via the queue into main with commit af1290c May 31, 2024
18 checks passed
@jorisdral jorisdral deleted the jdral/move-same-error branch May 31, 2024 16:58
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