-
Notifications
You must be signed in to change notification settings - Fork 6
chore: rename arbi-bom to orbi-bom #354
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
base: master
Are you sure you want to change the base?
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.
Pull Request Overview
This PR updates email addresses and team references from "arbi-bom" to "orbi-bom" in the Python requirements upgrade workflow to ensure notifications are sent to the correct team.
- Updates team reviewer assignment in pull request creation
- Changes failure notification email recipient
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
e4915c2
to
31ff33f
Compare
31ff33f
to
e5c5f49
Compare
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.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
- name: Send failure notification | ||
if: ${{ failure() }} | ||
- name: Send notification |
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.
The step name was changed from 'Send failure notification' to 'Send notification', but the conditional 'if: ${{ failure() }}' was removed. This means the notification will now be sent on every workflow run (success or failure) instead of only on failures.
Copilot uses AI. Check for mistakes.
Description
Update GitHub workflow failure email notifications to use orbi-bom@edx.org instead of arbi-bom@edx.org
Updates team reviewer assignment in pull request creation from arbi-bom to orbi-bom
Related PRs
edx/edx-arch-experiments#1107
edx/devstack#174
https://github.com/edx/edx-internal/pull/13377
https://github.com/edx/repo-health-data/pull/312
https://github.com/edx/jenkins-job-dsl-internal/pull/962
Jira - BOMS-39