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

Grammar etc #81

Merged
merged 4 commits into from
May 11, 2023
Merged

Grammar etc #81

merged 4 commits into from
May 11, 2023

Conversation

origamifreak2
Copy link
Contributor

I went through the documentation and updated some grammar, punctuation, and other things.

@netlify
Copy link

netlify bot commented May 11, 2023

Deploy Preview for stupefied-edison-1dae54 ready!

Name Link
🔨 Latest commit 8a108a4
🔍 Latest deploy log https://app.netlify.com/sites/stupefied-edison-1dae54/deploys/645c63bcd3bae400088b4307
😎 Deploy Preview https://deploy-preview-81--stupefied-edison-1dae54.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@origamifreak2
Copy link
Contributor Author

Also, I do have a question. I noticed but the markdown documentation seems to be wrapping itself after a certain number of columns. What is the number of columns you're trying to limit it to?

@sgenoud
Copy link
Owner

sgenoud commented May 11, 2023

Thanks for all these fixes! As you can guess English is not my first language (and I tend to get lazy after writing for a while). These changes will make the project more appealing!

What is the number of columns you're trying to limit it to?

I think it is 79 or 80 - but don't mind it, it is mostly a side effect to the fact that I use vim which tends to make it nicer to work with hard wraps. But I don't mind having some mixed stuff in there!

@sgenoud sgenoud merged commit 6113981 into sgenoud:main May 11, 2023
5 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators May 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants