Skip to content
This repository was archived by the owner on Aug 27, 2022. It is now read-only.

Minor typographical changes - #52

Merged
javcasas merged 2 commits into
connecting_haskell_backend_to_purescript_frontendfrom
jsl-minor-changes
Jul 13, 2017
Merged

Minor typographical changes#52
javcasas merged 2 commits into
connecting_haskell_backend_to_purescript_frontendfrom
jsl-minor-changes

Conversation

@jsl

@jsl jsl commented Jul 12, 2017

Copy link
Copy Markdown
Member

No description provided.

@javcasas javcasas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes, my English becomes rusty if I don't oil it often enough.


Well, there is a way to do that. Can you see the `Generic` instances we have introduced somehow?
We are going to leverage these instances for us. In fact, I have not invented this, the [Argonaut](https://github.com/purescript-contrib/purescript-argonaut) guys did it!
We are going to take advantage of these instances. In fact, I have not invented this - the [Argonaut](https://github.com/purescript-contrib/purescript-argonaut) guys did it!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is it worth it changing the guys to folks there?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good question. If you know for a fact that all of the people on the team were men, I think it's ok to say "guys" - otherwise we should probably make it more gender-neutral.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Or simply say, "team."

@javcasas
javcasas merged commit 1ebfab5 into connecting_haskell_backend_to_purescript_frontend Jul 13, 2017
@javcasas
javcasas deleted the jsl-minor-changes branch July 13, 2017 22:10
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