-
Notifications
You must be signed in to change notification settings - Fork 1
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
Style guide (code / syntax) #49
Comments
@paulyoung This is on my personal todo! I have concrete ideas if you'd like to coordinate? I was thinking of publishing it with GitBook for a whirlwind syntax tour. |
Sounds great! I may be providing more questions than answers 😄 |
Sweet I'll gather my thoughts into a repo today!! |
@paulyoung I'm starting here if you wanna follow along. I'll publish conventions first based on several Haskell and Elm guides. NB for pedagogical sake it won't follow the style of our Prelude or compiler. |
@AppShipIt sounds good. Feel free to ping me if you make updates. |
I think this should go hand-in-hand with #23. |
I think #23 should be more flexible. At work I've pointed out the current conventions I deem awful e.g. vertical indentation. We're using recommendations based on Elm which I'll add to purescript-4humans. |
What's "vertical indentation"? |
Next: purescript-4dolphins 😉 |
data Maybe a = Nothing
| Just a
-- ^ Lining this up is "vertical indentation". Edit: ... I think. |
The Elm style guide is against that too I thought? Unless I'm misunderstanding @AppShipIt. 👍 for not doing that. |
Right. Definitely planning on getting feedback! |
👍 for this. I've also been using the Elm style guide for PS at work and it's been very nice. |
I started work on one here, if anyone is interested. |
Moving to the docs repo. |
Not to be confused with the guidelines for writing libraries, but something more in the style of Elm's style guide or NoRedInk's Elm Style Guide.
The text was updated successfully, but these errors were encountered: