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 spelling fixes. #1

Merged
merged 2 commits into from
Apr 28, 2022
Merged

Conversation

zoq
Copy link
Contributor

@zoq zoq commented Apr 18, 2022

Minor spelling fixes, my editor auto-formatted some of the lines, let me know if I should revert the 80 column character formation.

Signed-off-by: Marcus Edel [email protected]

Signed-off-by: Marcus Edel <[email protected]>
@zoq zoq added the documentation Improvements or additions to documentation label Apr 18, 2022
README.md Outdated
@@ -4,12 +4,16 @@
<video src="https://user-images.githubusercontent.com/107984/157506173-5d2788e1-71ca-49ae-afc3-b7f7afbab798.mp4"></video>
</div>

*We asked an [AI](https://colab.research.google.com/github/zippy731/disco-diffusion-turbo/blob/main/Disco_Diffusion_v5_Turbo_%5Bw_3D_animation%5D.ipynb) what it thinks about our SocketCAN Kubernetes plugin, this was it's answer...
*We asked an [AI](https://colab.research.google.com/github/zippy731/disco-diffusion-turbo/blob/main/Disco_Diffusion_v5_Turbo_%5Bw_3D_animation%5D.ipynb) what it thinks about our SocketCAN Kubernetes plugin, this was it's answer...
Copy link
Contributor

Choose a reason for hiding this comment

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

The double-space removal at the end is unintended, right?

For myself I disabled the automatic space removal at the end of line and instead installed a plugin that just highlights these spaces so I can decide myself. I guess disabling it for Markdown files would work as well.

@jpc
Copy link
Contributor

jpc commented Apr 27, 2022

The 80-character wrapping (or the justification I once used) makes it nicer to read in text form but completely ruins the diffs. :/

@zoq
Copy link
Contributor Author

zoq commented Apr 28, 2022

The 80-character wrapping (or the justification I once used) makes it nicer to read in text form but completely ruins the diffs. :/

I reverted the format changes and just committed the minor spelling fixes.

@jpc jpc merged commit f654859 into collabora:main Apr 28, 2022
@jpc
Copy link
Contributor

jpc commented Apr 28, 2022

Thanks a lot :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Development

Successfully merging this pull request may close these issues.

2 participants