copilot icon indicating copy to clipboard operation
copilot copied to clipboard

Compliance with style guide (partial) (2)

Open ivanperez-keera opened this issue 2 years ago • 0 comments

Description

In our effort to move Copilot to Class D, we have to comply with a style guide. In order to progressively follow those rules, we split complying with the style guide into a number of smaller style-related issues.

This is the second issue pertaining to style changes for conformance with Class D.

Type

  • Management: conformance with new requirement.

Additional context

  • See #316 for other style changes addressed.

Requester

  • Ivan Perez

Method to check presence of bug

Not applicable (not a bug).

Expected result

All Copilot code conforms to the following rules:

  • All exports should be explicit.
  • All imports should be explicit.
  • Imports should be grouped, sorted, alphabetized, and aligned.
  • The standard indendation is 2 spaces.
  • Always leave a space after a comma.
  • Do not require unnecessary pragmas.
  • Do not add unnecessary imports.
  • Do not include redundant brackets, except where the brackets are meant to communicate a conceptual unit.
  • Do not include redundant $.
  • Whenever possible, prefer simpler solutions and expressions.
  • Use haddock formatting to capture copyright notices.
  • Prefer guards to multiple nested if-then-else.

Desired result

All code complies with the rules above.

Proposed solution

Make all code comply with the rules above.

Further notes

None.

ivanperez-keera avatar May 13 '22 00:05 ivanperez-keera