Skip to content

[llvm][Docs] add discord bot calendar capability notes #122140

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

Merged
merged 1 commit into from
Jan 9, 2025

Conversation

gburgessiv
Copy link
Member

The LLVM Discord bot now has the ability to scrape the LLVM calendar & send reminders about upcoming office hours events and sync-ups. Document that here.

While I'm in the area, add a note about the bot's ability to @mention people when they're on buildbot blamelists.

Related to llvm/Community.o#19

Copy link
Collaborator

@AaronBallman AaronBallman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for working on this! I think it looks good, but please wait a bit before landing in case anyone else has feedback.

Copy link
Collaborator

@kbeyls kbeyls left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall.

I just have 2 suggestions for a bit more text:

  1. in the comment I did inline.
  2. Add some text to the section https://llvm.org/docs/GettingInvolved.html#guidance-on-what-to-put-into-llvm-community-calendar-invites, as probably that's the place where people look for guidance when setting up a new event. I'd consider adding an example meta-data text block at the end of the example there, as I assume many new organizers might simply copy paste that example and then adapt it. By doing so, we'd make it a lot more likely for this bot functionality to be used by most events.

The LLVM Discord bot now has the ability to scrape the LLVM calendar &
send reminders about upcoming office hours events and sync-ups. Document
that here.

While I'm in the area, add a note about the bot's ability to @mention
people when they're on buildbot blamelists.

Related to llvm/Community.o#19
@gburgessiv gburgessiv force-pushed the add-discord-bot-capability-notes branch from d4d6bcc to dddfc63 Compare January 8, 2025 19:46
@gburgessiv
Copy link
Member Author

Add some text to the section https://llvm.org/docs/GettingInvolved.html#guidance-on-what-to-put-into-llvm-community-calendar-invites, as probably that's the place where people look for guidance when setting up a new event

Huh, I didn't see that section. Agreed that it'd be good to add stuff there; PTAL!

@kbeyls
Copy link
Collaborator

kbeyls commented Jan 8, 2025

Add some text to the section https://llvm.org/docs/GettingInvolved.html#guidance-on-what-to-put-into-llvm-community-calendar-invites, as probably that's the place where people look for guidance when setting up a new event

Huh, I didn't see that section. Agreed that it'd be good to add stuff there; PTAL!

Thanks, fully LGTM now!

@gburgessiv
Copy link
Member Author

Thanks for the reviews!

Ignoring buildkite failure because this change shouldn't impact LLVM.tools/llvm-gsymutil/ARM_AArch64/macho-merged-funcs-dwarf.yaml :)

@gburgessiv gburgessiv merged commit 2420361 into llvm:main Jan 9, 2025
7 of 9 checks passed
@gburgessiv gburgessiv deleted the add-discord-bot-capability-notes branch January 9, 2025 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants