Skip to content

Use base-compat in tests#1607

Merged
paf31 merged 2 commits intopurescript:masterfrom
phadej:base-compat-tests
Nov 5, 2015
Merged

Use base-compat in tests#1607
paf31 merged 2 commits intopurescript:masterfrom
phadej:base-compat-tests

Conversation

@phadej
Copy link
Copy Markdown
Contributor

@phadej phadej commented Nov 4, 2015

First step for #1604

This touches only tests, to act as an example. I could convert the rest either in single big PR or few smaller.

@phadej phadej force-pushed the base-compat-tests branch from e70004a to 621a5a7 Compare November 4, 2015 11:29
@paf31
Copy link
Copy Markdown
Contributor

paf31 commented Nov 4, 2015

Looks nice, thanks! Does license-generator change anything if the changes are only in executables? I think it should.

@phadej
Copy link
Copy Markdown
Contributor Author

phadej commented Nov 4, 2015

Let's see, cabal-dependency-licenses wants cabal configure :/ compiling dependencies atm.

@phadej
Copy link
Copy Markdown
Contributor Author

phadej commented Nov 4, 2015

ping @paf31

@paf31
Copy link
Copy Markdown
Contributor

paf31 commented Nov 4, 2015

👍 Looks fantastic, thanks. I'll merge later this evening.

@phadej phadej force-pushed the base-compat-tests branch from c8ce4d0 to 8f7583e Compare November 4, 2015 16:55
@phadej phadej mentioned this pull request Nov 5, 2015
@paf31 paf31 merged commit 8f7583e into purescript:master Nov 5, 2015
@phadej phadej deleted the base-compat-tests branch November 5, 2015 17:35
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