Closed kierisi closed 9 months ago
pre-commit.ci autofix
@kierisi You have some trailing whitespace in the markdown file and pre-commit is grumpy and won't autofix.
@lwasser @kierisi Let's add a DO-NOT-MERGE label to this repo and label this PR 🛑
Enthusiastically approve. We are so lucky to have you here.
thank you! I'm so happy to be here 👯
@kierisi You have some trailing whitespace in the markdown file and pre-commit is grumpy and won't autofix.
thanks! I'll get that fixed -- old R habits die hard 😅
pre-commit.ci autofix
😆 i've NEVER seen pre-commit give a thumbs down @willingc @kierisi i am laughing at it's grumpiness. sometimes i think the bot gets tied up or has issues. i mean don't we all have those types of days ?
@willingc i have this theory that pre commit is unhappy because you didn't have permissions to maintain this repo. but i just added you. let's see in the future if it gives you a rocket emoji instead of an unhelpful if not RUDE 😆 thumbs down. i mean seriously a thumbs down for someone that is trying to contribute and help doesn't seem right. 🙃 this is good to go. the one x is related to this being a new page on the website and i still haven't had time to figure out how to tell htmlproofer to check for links against the built site not against the live site.
shall we merge now or @willingc did you want to pause on the merge for some other reason other than CI grumpiness?
@kierisi you should also be able to run pre-commit.ci autofix
so you don't have to deal with pre-commit stuff locally if you don't want to. we should test that you have correct permissions in general as well! both you and carol!
Thanks @lwasser. I will let @kierisi or you merge :D
merging this so it's available online!!
Community Manager intro post - please note that the post is dated for 2023-11-29, and shouldn't be published before then :) Back-dating the post will allow for a local preview of the post.