-
Notifications
You must be signed in to change notification settings - Fork 344
N17: macroquad #424
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
N17: macroquad #424
Conversation
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.
Content looks good, but there's a few formatting things tripping up the CI build:
content/posts/newsletter-017/index.md:115:103 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
content/posts/newsletter-017/index.md:115:81 MD013/line-length Line length [Expected: 80; Actual: 103]
content/posts/newsletter-017/index.md:117:267 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
content/posts/newsletter-017/index.md:117:81 MD013/line-length Line length [Expected: 80; Actual: 267]
content/posts/newsletter-017/index.md:119:209 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 3]
content/posts/newsletter-017/index.md:119:81 MD013/line-length Line length [Expected: 80; Actual: 211]
You're also missing the URL for your miniquad
link, by the looks of it.
f24ad70
to
12ae49e
Compare
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.
👍 Looks good!
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.
👍 (made a few style & fmt tweaks)
Merged, thanks! |
No description provided.