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

Add the polishing changes from #435 in their own PR #442

Merged
merged 4 commits into from
Feb 9, 2022

Conversation

cansavvy
Copy link
Collaborator

@cansavvy cansavvy commented Feb 9, 2022

Purpose/implementation Section

What changes are being implemented in this Pull Request?

There's a bunch of little polishing changes that don't really belong to the main point of #435 and don't require careful review, so I'm pulling those out into their very own PR.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2022

No spelling errors! 🎉
Comment updated at 2022-02-09 with changes from 87c94e8

@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2022

Re-rendered previews from the latest commit:

Updated at 2022-02-09 with changes from 1be5516

@cansavvy
Copy link
Collaborator Author

cansavvy commented Feb 9, 2022

None of these changes are huge, and for the purposes of moving #435 forward and making it cleaner, I'm going to merge this if the checks pass.

@cansavvy
Copy link
Collaborator Author

cansavvy commented Feb 9, 2022

All set

@cansavvy cansavvy merged commit 4a96ab3 into main Feb 9, 2022
@cansavvy cansavvy deleted the cansavvy/polishing branch February 9, 2022 16:05
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