tripleee / sloshy

Sloshy the Thawman, a simple chatbot to keep rooms alive on Stack Overflow / Stack Exchange
6 stars 11 forks source link

Added 24 chat rooms to sloshy.yaml #2

Closed ndattani closed 3 years ago

ndattani commented 3 years ago

I assumed the name doesn't have to exactly match the chatroom's URL because name: "sloshy" doesn't exactly match the chatroom title Sloshy the Thawman, but I can certainly change all my name fields if necesasry.

tripleee commented 3 years ago

Thanks for your contribution! I'm hesitant to add this many rooms without an opinion from room owners or site moderators, though. Could you please ask for someone with suitable authority to weigh in on this PR, or perhaps at least document room ownership (I assume most of them are yours?)

tripleee commented 3 years ago

Also, as a matter of simple aesthetics, could you please trim the empty lines from the end and rebase? Thanks.

ndattani commented 3 years ago

I think I'm the room owner of all of the rooms, but let's see if @TyBalduf (a moderator at MMSE) can reply with explicit approval. I'll remove the empty lines from the end though!

TyBalduf commented 3 years ago

I'm a mod for Matter Modeling SE. I'm fine with seeing all these rooms added. We are trying to maintain rooms for specific topics, but because the site is still pretty young, there are times where we have a lack of activity. This will save us the trouble of having to go through and comment or unfreeze them.

I did notice that QChem is on the list twice though.

tripleee commented 3 years ago

Thanks for the fixes and the followup. Could you still remove the duplicate noted by @TyBalduf please? Also, a rebase to a single commit (or maybe two commits) would be nice.

tripleee commented 3 years ago

Also, for what it's worth, Sloshy does not yet have enough rep on meta, but I'm working furiously on that (-:

ndattani commented 3 years ago

I hadn't seen that added point about QChem being listed twice, since that came in an edit to the original comment. @TyBalduf I typically use edits to fix typos and such, and write a new comment altogether if adding new information (like tripleee did in the last 2 comments). If there's no new comments since the last visit, a third-party observer won't easily know that something's been added when they re-visit the page.

I've removed the duplication of QChem.

I ran git rebase -i HEAD~3 and squashed the most recent 3 commits into 1 commit, so the following 3 commits are all in one:

1) removal of whitespace at the end of the file 2) removal of duplicate QChem entry 3) Adding chat.meta.stackexchange.com (I see you gained rep because the edits were approved!)

tripleee commented 3 years ago

Thanks for the update. But now the rebase seems wrong, it wants to add six commits instead of one?

I didn't have enough rep on meta until this morning but this is now good to go if you can fix the rebase. Alternatively, if you prefer, I can try to finish it up.

ndattani commented 3 years ago

About the rebase, I ran the command git rebase -i HEAD~3 to prepare for the squash of the most recent 3 commits. In the screen that popped up, I replaced pick by s for the last 2/3 commits, so that they would be "squashed" with the 1st of the 3 commits. This combined 3 commits into one, so I thought that my 4 commits would become just 2 commits. I followed these instructions.

But strangely I couldn't push this without first doing a merge (even though I was completely up-to-date with the remote repo, because I had just cloned it before starting to do the rebase/squash). This merge brought my branch up to 5 commits, then the squashed commit which combined 3 commits into 1, was added as an additional commit instead of replacing three commits by 1, and this brought me up to 6 commits when I was only expecting it to be 2 in total.

Yes if you're able to finish it up, that would be nice.

tripleee commented 3 years ago

I have often found that I need to git pull --rebase master after a rebase to realign the commit history. Maybe this should be reported as a bug in Git; I don't know. Anyway, I can take it from here.