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

Support th-abstraction 0.6 #495

Merged
merged 1 commit into from
Sep 21, 2023
Merged

Support th-abstraction 0.6 #495

merged 1 commit into from
Sep 21, 2023

Conversation

Vekhir
Copy link
Contributor

@Vekhir Vekhir commented Sep 14, 2023

Builds and tests fine

@adamgundry
Copy link
Member

Thanks for the patch! I ran CI and it looks like there is a glitch in the doctests on GHC <= 9.2.8 (but this might well not be due to your change...).

src/Data/HashSet/Optics.hs:36: failure in expression `setOf folded ["hello","world"]'
expected: fromList ["hello","world"]
 but got: fromList ["world","hello"]
                     ^

Examples: 97  Tried: 96  Errors: 0  Failures: 1

@phadej
Copy link
Contributor

phadej commented Sep 15, 2023

but this might well not be due to your change

That's most likely a change in hashable, which doesn't guarantee hash values to stay stable, so toList of HashSet may list values in any order. After all, it's defined in unordered-containers package!

@adamgundry adamgundry merged commit 646db47 into well-typed:master Sep 21, 2023
3 of 10 checks passed
@phadej
Copy link
Contributor

phadej commented Sep 21, 2023

FWIW, e.g. GHC-9.6.2 job picked up th-abstraction-0.5.0.0 (lib) (requires download & build) (check dump install plan build step).

Testing locally, the reason is:

[__4] rejecting: th-abstraction-0.6.0.0 (conflict: generics-sop =>
th-abstraction>=0.4 && <0.6)

... so this PR seems to be merged prematurely.


EDIT: also

[__5] rejecting: template-haskell-optics-0.3 (conflict:
th-abstraction==0.6.0.0, template-haskell-optics => th-abstraction>=0.4 &&
<0.6)

@adamgundry
Copy link
Member

I've tested locally that optics-th builds and passes tests with th-abstraction-0.6.0.0 (using allow-newer). Is that enough or should we revert?

template-haskell-optics seems to be hit by an API change in th-abstraction-0.6.0.0, so it doesn't build.

@phadej
Copy link
Contributor

phadej commented Sep 22, 2023

I've tested locally that

Please mention these, that would prevent confusion.

should we revert?

No need to, as long as someone remembers to make template-haskell-optics compatible as well.

@adamgundry
Copy link
Member

Got it.

I've bumped the bounds of https://hackage.haskell.org/package/optics-th-0.4.1 to permit th-abstraction-0.6.

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