Skip to content

Conversation

@ReedMerrill
Copy link
Contributor

This change involves a variety of edits to improve readability, including removing repetitive text, improving spelling and grammar, and improvements to convey the same meaning in fewer words.

I started with contribute.md to see if this is the sort of readability improvements you are looking for. I'll be happy to work on more long documentation like this too and I'm looking forward to your feedback.

Signed-off-by: Reed <reedjmerrill@gmail.com>
@netlify
Copy link

netlify bot commented Jul 26, 2025

Deploy Preview for mlsysops ready!

Name Link
🔨 Latest commit 50ebadc
🔍 Latest deploy log https://app.netlify.com/projects/mlsysops/deploys/6884374fc72ae50008ae2207
😎 Deploy Preview https://deploy-preview-40--mlsysops.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 project configuration.

@ananos
Copy link
Contributor

ananos commented Jul 26, 2025

Hi @ReedMerrill!

thanks very much for your patch! Looks good, and is inline with what we had in mind. I'm adding the necessary labels to allow the CI to run, please feel free to coordinate with @RR-Sahoo at #37 so that we can avoid merge conflicts.

thanks!

@ReedMerrill
Copy link
Contributor Author

@ananos Please let me know if there is anything else I need to do for this PR!

@ananos
Copy link
Contributor

ananos commented Aug 2, 2025

thanks @ReedMerrill I'm keen to merge this as it fixes stuff. However there's more work to be done in the contributing document so feel free to tackle this on a follow-up PR.

thanks so much for your contributions!

@ananos ananos merged commit b83e2bd into mlsysops-eu:main Aug 2, 2025
23 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants