Skip to content
This repository has been archived by the owner on Jul 25, 2018. It is now read-only.

Updating to be compliant with current Fantasy Land spec #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mwalkerwells
Copy link

While unfortunate, tests are being removed until dependent libraries (fantasy-check, fantasy-helpers, fantasy-identities) are also updated to be compliant with current Fantasy Land spec.

@@ -1,6 +1,6 @@
{
"name": "fantasy-states",
"version": "0.2.1",
"version": "0.2.2",
Copy link
Member

Choose a reason for hiding this comment

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

Let's not update this until release time. Also, as this is a breaking change the next release should be (at least) v0.3.0.

@@ -2,17 +2,18 @@

const daggy = require('daggy');

const {of, chain, map, ap} = require('fantasy-land');
Copy link
Member

Choose a reason for hiding this comment

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

I don't see fantasy-land listed as a dependency.

@davidchambers
Copy link
Member

While unfortunate, tests are being removed until dependent libraries (fantasy-check, fantasy-helpers, fantasy-identities) are also updated to be compliant with current Fantasy Land spec.

It seems we should work our way here from the leaf nodes. :)

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.

2 participants