Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Adjust notifications for documentation work #27843

Merged
merged 1 commit into from Sep 9, 2022
Merged

Adjust notifications for documentation work #27843

merged 1 commit into from Sep 9, 2022

Conversation

ebullient
Copy link
Contributor

No description provided.

@quarkus-bot quarkus-bot bot added the area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure label Sep 9, 2022
@gsmet gsmet merged commit ea26f96 into main Sep 9, 2022
@quarkus-bot quarkus-bot bot added this to the 2.13 - main milestone Sep 9, 2022
@gsmet
Copy link
Member

gsmet commented Sep 9, 2022

@ebullient so you confirm pinging a team doesn't work?

@ebullient
Copy link
Contributor Author

It seemed not to, per @inoxx03.

@gastaldi gastaldi deleted the ebullient-patch-1 branch September 10, 2022 23:55
inoxx03
inoxx03 reviewed Sep 12, 2022
View reviewed changes
.github/quarkus-github-bot.yml
@@ -387,7 +387,7 @@ triage:
- independent-projects/bootstrap/
- independent-projects/tools/
- labels: [area/documentation]
notify: [ebullient, quarkusio/Documentation]
notify: [ebullient, michelle-purcell, hmanwani-rh, sheilamjones, MichalMaler]
Copy link

Choose a reason for hiding this comment

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

@ebullient Would you please also add Sunayna (sunayna15) and me (inoxx03) to the notify list? For general awareness, triage, and writer backup in case some of the writers are on PTO when they get notified. Thank you:)

Copy link
Contributor Author

@ebullient ebullient Sep 12, 2022 •
edited

Choose a reason for hiding this comment

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

HA. Somehow I forgot you?! :sigh: Where are my brains .. See #27866

@inoxx03
Copy link

inoxx03 commented Sep 12, 2022

It seemed not to, per @inoxx03.

@gsmet @ebullient: I can confirm this, too. Writers say that they haven't had any notifications about docs changes in this repo come through to their inbox yet. AFAIK, they definitely should have been receiving them, since new issues for area/documentation have been popping up on the docs board.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@inoxx03 inoxx03 inoxx03 left review comments

@gastaldi gastaldi gastaldi approved these changes

@gsmet gsmet gsmet approved these changes

Assignees
No one assigned
Labels
area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure
Projects
None yet
Milestone
2.13.0.CR1
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@ebullient @gsmet @inoxx03 @gastaldi