-
Notifications
You must be signed in to change notification settings - Fork 109
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
Decide on style for [def]rule_tac ... [and ...] in ...
#746
Comments
I 'm fine the first two (pretty left, standard right), and can live with the naive left version as well, so those are all good. I guess variant 1 and 2 at the end just come down to being the degenerate forms of the left and right versions, so both should be permitted, and it also explains why Raf prefers variant 1 and I prefer variant 2 :-) So we really just have the 2 preferred versions, and the naive-left version in addition to that. |
You are right that the degenerate form results in an ambiguous guideline, but I guess we can't win them all. Ok, then TODO for me is to schreib it in the style guide. |
I agree with everything said here, and can add another data point as someone who prefers the original left-operator pretty version 1 and then prefers version 1 of the one-variable case. One additional comment on the right-operator version; I would be against the |
Thanks @corlewis !
Checking to confirm, since none of the above examples have the |
I meant line and was referring to the comment here
|
Agreed, I would probably also not put the Let's remove the comment. |
Right, I see it now. Agreed with both of you, have changed the comment above. |
Added in #834 |
We had some discussions on the Proofcraft mattermost (present were myself, @lsf37 , @corlewis , @michaelmcinerney ), which ended up with things not quite nailed down in the end. Let's nail it down and put it in the style guide.
General considerations:
and
) and right-operator (and
) versionin
belong to therule_tac
, or the instantiation? (in previous discussions: to therule_tac
)and/in
to utilise space better (similar to:
alignment on method args)At the time of discussion, we settled roughly on:
The following were ruled out:
However, the above consensus caused friction in the following (one-variable long instantiation) case (pointed out by @lsf37), bringing us back to a need to go over this and pin things down:
For recent proofs I've been going with version 1, because of its consistency with pretty version 1 above.
One of these days maybe we'll have a tool, but even then we'd still have to decide what the tool reformats to.
The text was updated successfully, but these errors were encountered: