-
Notifications
You must be signed in to change notification settings - Fork 13
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
ENH: Convert team syncs from issues to slack only #347
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.
This looks good to me.
Two small things I noticed though:
- the bot doesn't send the messages in the morning
- should we give more details about the Slack updates? I noticed there isn't any info about the M-W-F workflow. But maybe this would make sense in the TODO section.
@GeorgianaElena could you go into more detail on your second point? What extra information did you have in mind, and where? I'm happy to add in some extra detail somewhere! |
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.
LGTM
Another ping for @GeorgianaElena to provide some guidance on what you mean by "give more details about the Slack updates". I'm happy to make changes if you wish, just not sure where you'd like them to be! If you don't think it's a blocker or don't have time to look by tomorrow, I'll merge this one in and we can always iterate |
@choldgraf, sorry for my message being confusing 🤦 When I said "more details" I was referring to mentioning that the updates happen M-W-F. Definitely not a must or a blocker. Sorry for the confusion again! |
Ah thanks for that clarification @GeorgianaElena - I've added a sentence to convey this information as well! Will merge now :-) |
This converts our team sync process to use the Slack Geekbot instead of GitHub issues each week. Here are major changes:
ref #259