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

It is not well thought-out, delete the repo #2

Open
AlexDaniel opened this issue Aug 31, 2017 · 4 comments
Open

It is not well thought-out, delete the repo #2

AlexDaniel opened this issue Aug 31, 2017 · 4 comments

Comments

@AlexDaniel
Copy link

AlexDaniel commented Aug 31, 2017

I know that this repo is old, but it was mentioned in a recent discussion so that's why I'm here.

There are two points in the document and both of them are either wrong or incomplete.

“Do not base your design on junctions and autothreading”

debugging code with junctions is hard

???

and often confusing to read

???

“Use whitespace around infix operators”

5 .. 10? 2 ** 32? base 255 : 16? Maybe even 42 . sqrt? That's not how people write these things (normally), and I don't see a convincing argument for a style guide to mandate these variants. That's without even mentioning the huge list of cases where having no whitespace can't cause any issue (e.g. all non-texas infix ops).

Having whitespace around some infix ops is good. But “It is therefore recommended to always use whitespace on both side of an infix operator.” is a statement that is too strong. And I wouldn't want want to see a huge list of exceptions there.

o/

@AlexDaniel
Copy link
Author

By the way, we have a list of recommendations that achieves a much better goal: https://docs.perl6.org/language/traps (also some pending sections).

@AlexDaniel
Copy link
Author

In fact, maybe we can even add “Infix operators without whitespace” point there.

@AlexDaniel
Copy link
Author

AlexDaniel commented Aug 31, 2017

Ah, we already have it. It does not mention 1<2 case, which we now have an issue for: Raku/doc#1481

@AlexDaniel
Copy link
Author

FWIW another option is to give proper explanations. For example: https://irclog.perlgeek.de/perl6/2017-10-22#i_15336776

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

No branches or pull requests

1 participant