Skip to content

Added Reformatting Guide #314

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

Merged
merged 11 commits into from
Apr 28, 2025
Merged

Added Reformatting Guide #314

merged 11 commits into from
Apr 28, 2025

Conversation

austincondiff
Copy link
Collaborator

Description

This will add a reformatting guide that is disabled by default. When enabled users will see a line drawn at a configurable column to guide them as to when and how they should reformat their code.

Related Issues

Checklist

  • I read and understood the contributing guide as well as the code of conduct
  • The issues this PR addresses are related to each other
  • My changes generate no new warnings
  • My code builds and runs on my machine
  • My changes are all related to the related issue above
  • I documented my code

Screenshots

image image image

@austincondiff austincondiff force-pushed the feat/reformatting-guide branch from 66d2af9 to 9c0e465 Compare April 24, 2025 20:45
Copy link
Collaborator

@avdept avdept left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM from Quick Look

Copy link
Member

@tom-ludwig tom-ludwig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few tips for improving the comments — other than that, great changes!

@austincondiff
Copy link
Collaborator Author

This is ready for review. I've fixed tests, SwiftLint issues, and resolved issues raised.

Copy link
Member

@tom-ludwig tom-ludwig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect!

@austincondiff austincondiff merged commit 3dccebd into main Apr 28, 2025
2 checks passed
@austincondiff austincondiff deleted the feat/reformatting-guide branch April 28, 2025 19:00
@tom-ludwig
Copy link
Member

Would this close #207?

@austincondiff
Copy link
Collaborator Author

No, invisible characters are completely different. They look like this:

image

@tom-ludwig
Copy link
Member

Oops I made a typo i meant #270 of course 😆

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.

4 participants