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

Add everything! #1

Merged
merged 1 commit into from
Jul 12, 2018
Merged

Add everything! #1

merged 1 commit into from
Jul 12, 2018

Conversation

Avaq
Copy link
Contributor

@Avaq Avaq commented Jul 11, 2018

No description provided.

@Avaq Avaq requested a review from paolobueno July 11, 2018 16:38
@Avaq
Copy link
Contributor Author

Avaq commented Jul 11, 2018

At the time of writing, coveralls is down, so we'll have to wait to see if reports are working.

@Avaq Avaq force-pushed the avaq/everything branch 3 times, most recently from 867ce83 to 539ed74 Compare July 11, 2018 16:54
paolobueno
paolobueno previously approved these changes Jul 11, 2018
eq (typeof result.types) ('object');
eq (result.reducer ({}, {type: 'Warped/Test/foo', payload: 'test'}))
({actionPayload: 'test'});
eq (result.types) ({foo: 'Warped/Test/foo'});

Choose a reason for hiding this comment

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

The space before function params really makes it look like a different language!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's the trick. See sanctuary-js/sanctuary#438 :)

//. Usage in Node depends on `--experimental-modules`.
//. With older Node versions, use [std/esm][2].

// createType :: (String, String) -> String

Choose a reason for hiding this comment

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

Nice tooling for embedding md

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Avaq
Copy link
Contributor Author

Avaq commented Jul 12, 2018

There we go. With Coveralls working, this work should be ready for a merge.

@Avaq Avaq force-pushed the avaq/everything branch from 8d38faf to 18691d4 Compare July 12, 2018 07:52
paolobueno
paolobueno previously approved these changes Jul 12, 2018
@Avaq Avaq force-pushed the avaq/everything branch from ea31d95 to f52aa6d Compare July 12, 2018 08:58
@Avaq Avaq merged commit 530e3e2 into master Jul 12, 2018
@Avaq Avaq deleted the avaq/everything branch July 12, 2018 10:16
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