Skip to content

Conversation

@vgulati-apphelix
Copy link
Contributor

@vgulati-apphelix vgulati-apphelix commented Oct 30, 2025

Description:

As arbi-bom does not exist today, switching dependabot, reviewers and email to arch-bom

TICKET:
Resolves edx/edx-arch-experiments#1124

Reviewers:

  • tag reviewer

Merge checklist:

  • All reviewers approved
  • CI build is green
  • Version bumped
  • Changelog record added
  • Documentation updated (not only docstrings)
  • Commits are squashed

team_reviewers: "arbi-bom"
email_address: arbi[email protected]
team_reviewers: "arch-bom"
email_address: arch[email protected]
Copy link
Contributor

@robrap robrap Oct 31, 2025

Choose a reason for hiding this comment

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

@vgulati-apphelix: Let's switch this to JSM. Do you want to duplicate the Orbi-BOM setup in JSM?

@openedx/2u-arch-bom: I plan to use JSM for this. We can decide later if we don't want this duplicated for team_reviewers.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@robrap I did not find JSM integration with apt email. Can you please create a new integration so that we can use the email or suggest which existing one I can use? I don't have access to add new integration for arch-bom.

Copy link
Contributor

Choose a reason for hiding this comment

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

You are now an admin. See Slack message. Thanks.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks. this is updated now

interval: "weekly"
reviewers:
- "openedx/arbi-bom"
- "openedx/arch-bom"
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this is the team name: https://github.com/orgs/openedx/teams/2u-arch-bom. Same issue elsewhere in file.

@robrap robrap merged commit 41a02c0 into openedx:master Nov 12, 2025
9 checks passed
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.

Switch reviewer and email to arch-bom - openedx/edx-django-utils

2 participants