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

Consistency of Haskell code style - compress multiple white lines #214

Closed
ivanperez-keera opened this issue May 15, 2022 · 0 comments
Closed
Assignees
Milestone

Comments

@ivanperez-keera
Copy link
Owner

A number of optional style suggestions were not implemented in #200 (see #200 (comment)).

Vertical space across the implementation is inconsistent. Some modules leave two spaces between definitions, others leave only one.

Our style guide includes:

4.5.1 Vertical Whitespace: always recommended
Suggestion: A single blank line should appear:

  1. Between consecutive definitions.

It therefore would be appropriate to always reduce vertical separation to one line.

@ivanperez-keera ivanperez-keera added this to the (+1) milestone May 15, 2022
@ivanperez-keera ivanperez-keera self-assigned this May 15, 2022
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…#214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
ivanperez-keera added a commit that referenced this issue May 15, 2022
…. Refs #214.

Suggestion: A single blank line should appear between consecutive definitions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant