Skip to content

fix: Remove notification group that is failing the build#80

Merged
sarina merged 1 commit intomasterfrom
update-reviewers
Feb 16, 2023
Merged

fix: Remove notification group that is failing the build#80
sarina merged 1 commit intomasterfrom
update-reviewers

Conversation

@sarina
Copy link
Copy Markdown

@sarina sarina commented Feb 16, 2023

This group is failing the upgrade-python-requirements build and I keep getting notified about it. My solution is to remove the group. If someone would like to be notified, please make a PR to fix this issue. https://github.com/openedx/xblock-sql-grader/actions/runs/4190725541/jobs/7264421740 @feanil @UsamaSadiq @iamsobanjaved

@sarina sarina merged commit b081697 into master Feb 16, 2023
@sarina sarina deleted the update-reviewers branch February 16, 2023 19:19
@UsamaSadiq
Copy link
Copy Markdown
Member

@feanil could you please verify if arbi-bom team has been added as collaborators in the repo settings or not? From the failed build logs, it seems like this could be an issue. Since individual members are being tagged successfully but tagging the team as a whole fails.
Same workflow and the team name is working perfectly in other repos so I'm investigating the reason why it started failing in this repo.
Thanks.

@feanil
Copy link
Copy Markdown

feanil commented Feb 17, 2023

@UsamaSadiq arbi-bom was not a team on this repo, can you file a tcril-engineering request for this, we want to track all additions of access via tickets there so we can have a log of the reasons for access.

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