-
Notifications
You must be signed in to change notification settings - Fork 344
N23: "thRustEngine" demo #702
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
Conversation
adding thRustEngine to engine updates
…stEngine Update index.md
added screenshot of thrustengine
added screenshot
thrustengine- removed repitition, explanation of public crate
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please can you reformat this to meet the style guidelines? Primarily:
- There should be one blank line above and below each heading and image (no more or less).
- Lines should be 80 characters long at most.
Also, it looks like you've uploaded a screenshot to the root of the repo - can you delete this?
removed accidental screenshot (shouldn't be here in root)
reformatted for style guidelines.
trying to match style guidelines
cleanup reformating,punctuation
yikes ok I deleted the extraneous screenshot. Is it lines or sentances of 80chars? I've shortened and split them etc- is this closer? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's 80 per line rather than per sentence (just makes the diffs a bit easier to read). I was able to fix the last few bits myself, thanks for sorting the screenshot issues :)
awesome, thanks, i'll know for next time |
Merged, thanks for the PR! |
Newsletter 23 #677