Skip to content
This repository has been archived by the owner on Apr 4, 2022. It is now read-only.

Fix package dependencies to be fully specified #11

Merged
merged 1 commit into from
Aug 31, 2020
Merged

Fix package dependencies to be fully specified #11

merged 1 commit into from
Aug 31, 2020

Conversation

mbj
Copy link
Owner

@mbj mbj commented Aug 31, 2020

Acceptance

  • Green CI

@mbj mbj requested a review from epicallan August 31, 2020 17:21
package.yaml Outdated
- source-constraints
- template-haskell
- text
- base > 4.12 && < 14.15
Copy link
Collaborator

@epicallan epicallan Aug 31, 2020

Choose a reason for hiding this comment

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

The 14.15 on this line should probably be 4.15.

Can we also do greater or equal to >= 4.12 as a lower bound so that we can support 8.6 and above? Not a blocker, could make changes in another iteration.

Copy link
Owner Author

Choose a reason for hiding this comment

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

Unless we had a build on 8.6 that works I'd not want to declare 8.6 support. source-constraints etc do not support 8.6 at this point.

I'd rather spend my energy forwards than backwards.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Unless we had a build on 8.6 that works I'd not want to declare 8.6 support. source-constraints etc do not support 8.6 at this point.
I'd rather spend my energy forwards than backwards.

Makes sense.

@mbj mbj merged commit 5baa8ec into master Aug 31, 2020
@mbj mbj deleted the fix/package branch August 31, 2020 17:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants