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

Remove trailing whitespace from a variety of files #540

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

floatingatoll
Copy link
Contributor

#528 included fixes for trailing whitespace in a variety of files, which I imagine we should fix regardless of #528 itself. These don't seem to be caught by CI, so this probably will not need to be merged into the CI PR I'm expecting to end up creating out of several other PRs.

Reviewers, my biggest concern is that this ought to be a no-op, but if it will trigger lambda autodeploys or other unknown things, then that would be of concern.

Replaces #532.

Copy link
Member

@bkochendorfer bkochendorfer left a comment

Choose a reason for hiding this comment

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

🟥

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