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

Minor whitespace changes #2595

Merged
merged 2 commits into from
Feb 6, 2025
Merged

Minor whitespace changes #2595

merged 2 commits into from
Feb 6, 2025

Conversation

randomPoison
Copy link
Collaborator

  • Add extra lines in a couple places to make code examples more readable. In both these cases I'm trying to break the code into more logical groups, generally some statement followed by a println showing the result of that operation.
  • Split the first example of struct initialization across a couple of lines for readability. I personally struggle to read that statement when it's all one line, so I expect it'd be hard for students, and I always break it up like this in class.

@randomPoison
Copy link
Collaborator Author

oh wait, this formatting is enforced by the formatter. How do I tell it to ignore this block?

@randomPoison
Copy link
Collaborator Author

Ah, found it

@djmitche djmitche self-requested a review January 31, 2025 17:30
@randomPoison randomPoison merged commit 6a25d7b into main Feb 6, 2025
37 checks passed
@randomPoison randomPoison deleted the legare/more-whitespace branch February 6, 2025 17:52
michael-kerscher pushed a commit to michael-kerscher/comprehensive-rust that referenced this pull request Feb 7, 2025
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.

2 participants