Add CONTRIBUTING.md #897

Merged
ghallak merged 13 commits from ghallak/contributing into master 2022-08-05 00:35:48 +09:00
ghallak commented 2022-08-03 20:53:32 +09:00 (Migrated from gitlab.com)
No description provided.
zxq9 commented 2022-08-03 23:54:35 +09:00 (Migrated from gitlab.com)

Created by: radrow

If a PR

*Created by: radrow* If **a** PR
zxq9 commented 2022-08-03 23:55:29 +09:00 (Migrated from gitlab.com)

Created by: radrow

Mention fold and prettyprint that have to be updated too

*Created by: radrow* Mention fold and prettyprint that have to be updated too
ghallak commented 2022-08-04 00:32:37 +09:00 (Migrated from gitlab.com)

@radrow I just noticed that this was missing, do you think it's correct?

@radrow I just noticed that this was missing, do you think it's correct?
zxq9 commented 2022-08-04 00:36:27 +09:00 (Migrated from gitlab.com)

Created by: radrow

Check if it behaves nice with other ops, such as

2+2 |> somefun

2::[] ++ [] |> somefun

*Created by: radrow* Check if it behaves nice with other ops, such as 2+2 |> somefun 2::[] ++ [] |> somefun
ghallak commented 2022-08-04 00:37:32 +09:00 (Migrated from gitlab.com)

Do you know what the 3 numbers returned by the function mean?

Do you know what the 3 numbers returned by the function mean?
zxq9 commented 2022-08-04 01:27:27 +09:00 (Migrated from gitlab.com)

Created by: radrow

Check usage, but I would guess operator priority over pre-, in-, and suffix operators

*Created by: radrow* Check usage, but I would guess operator priority over pre-, in-, and suffix operators
ghallak commented 2022-08-04 01:34:23 +09:00 (Migrated from gitlab.com)

It looks like when using infixl, the first and second numbers are the same, and the third number is the next precedence. This looks correct.

It looks like when using `infixl`, the first and second numbers are the same, and the third number is the next precedence. This looks correct.
zxq9 commented 2022-08-04 02:50:10 +09:00 (Migrated from gitlab.com)

Created by: hanssv

I think it should be aesophia...

*Created by: hanssv* I think it should be aesophia...
zxq9 commented 2022-08-04 02:50:49 +09:00 (Migrated from gitlab.com)

Created by: hanssv

Review: Dismissed

👍

*Created by: hanssv* **Review:** Dismissed 👍
zxq9 commented 2022-08-04 05:25:27 +09:00 (Migrated from gitlab.com)

Created by: radrow

Just "Sophia" would be the safest

*Created by: radrow* Just "Sophia" would be the safest
ghallak commented 2022-08-04 06:35:38 +09:00 (Migrated from gitlab.com)

Done here 321edb0e7b

Done here 321edb0e7bdb70f8b3611245a519131aea9b70c9
zxq9 commented 2022-08-04 22:04:15 +09:00 (Migrated from gitlab.com)

Created by: radrow

Review: Approved

*Created by: radrow* **Review:** Approved
zxq9 commented 2022-08-05 00:34:50 +09:00 (Migrated from gitlab.com)

Created by: hanssv

Review: Approved

*Created by: hanssv* **Review:** Approved
zxq9 commented 2022-08-05 00:35:48 +09:00 (Migrated from gitlab.com)

Merged by: ghallak at 2022-08-04 15:35:48 UTC

*Merged by: ghallak at 2022-08-04 15:35:48 UTC*
Sign in to join this conversation.
No description provided.