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

cnf ran: improve style consistency across suites #252

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

klaskosk
Copy link
Collaborator

Makes a number of style changes to increase the consistency throughout the cnf/ran subdir. Some of the changes included:

  • Enforcing By messages begin with a lowercase letter and a gerund
  • Enforcing It messages begin with a conjugated third person verb and not "should"
  • Preventing extra indentation by breaking up strings with a plus
  • Removing trailing whitespace from blocks
  • Removing commented code
  • Removing unnecessary variable declarations
  • Preferring When over Context as a container node and updating messages

Makes a number of style changes to increase the consistency throughout the cnf/ran subdir. Some of the changes included:

* Enforcing By messages begin with a lowercase letter and a gerund
* Enforcing It messages begin with a conjugated third person verb and not "should"
* Preventing extra indentation by breaking up strings with a plus
* Removing trailing whitespace from blocks
* Removing commented code
* Removing unnecessary variable declarations
* Preferring When over Context as a container node and updating messages
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

Successfully merging this pull request may close these issues.

1 participant