Skip to content

Add void function#455

Merged
paf31 merged 2 commits intopurescript:masterfrom
pelotom:patch-1
Jun 28, 2014
Merged

Add void function#455
paf31 merged 2 commits intopurescript:masterfrom
pelotom:patch-1

Conversation

@pelotom
Copy link
Copy Markdown
Contributor

@pelotom pelotom commented Jun 28, 2014

No description provided.

@paf31
Copy link
Copy Markdown
Contributor

paf31 commented Jun 28, 2014

Prelude defines Unit and quite a lot of other libraries have started using it. Since {} can't have typeclass instances, we tend to prefer Unit over {}, so this looks great but I would use Unit instead.

@pelotom
Copy link
Copy Markdown
Contributor Author

pelotom commented Jun 28, 2014

Also I forgot to export it, oops

paf31 added a commit that referenced this pull request Jun 28, 2014
@paf31
Merge pull request #455 from pelotom/patch-1
4c7733c 
Add `void` function
@paf31 paf31 merged commit 4c7733c into purescript:master Jun 28, 2014
@paf31
Copy link
Copy Markdown
Contributor

paf31 commented Jun 28, 2014

Thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pelotom @paf31