Skip to content

chore: remove arbi-bom references#180

Merged
timmc-edx merged 3 commits intoopenedx:masterfrom
vgulati-apphelix:vgulati/BOMS-39
Dec 16, 2025
Merged

chore: remove arbi-bom references#180
timmc-edx merged 3 commits intoopenedx:masterfrom
vgulati-apphelix:vgulati/BOMS-39

Conversation

@vgulati-apphelix
Copy link

@vgulati-apphelix vgulati-apphelix commented Dec 15, 2025

Description
Remove arbi-bom references

Resolves 1170

@vgulati-apphelix vgulati-apphelix changed the title Vgulati/boms 39 chore: remove arbi-bom references Dec 15, 2025
@vgulati-apphelix
Copy link
Author

I am not sure if email - "2u-arch-bom@edx.org" exists. Need this confirmation.

@timmc-edx
Copy link

timmc-edx commented Dec 15, 2025

I don't think it does. But thinking on this more, shouldn't dependency-related alerts be going to Orbi-BOM? Like this: https://github.com/edx/edx-arch-experiments/blob/0b959f3976b2be7ea3dfe328f4a64905f1914498/.github/workflows/upgrade-python-requirements.yml#L18

@vgulati-apphelix
Copy link
Author

@timmc-edx
I am not sure if having orbi-bom as reviewer will work as orbi-bom is not cc and wont have merge rights for these PRs. Not sure if you are suggesting something different here.

@timmc-edx
Copy link

@vgulati-apphelix
Copy link
Author

@timmc-edx changes are in there, can you please approve and merge?

@timmc-edx
Copy link

Looks good. We'll have to look into this dependency/test issue at some point though...

@timmc-edx timmc-edx merged commit 8b54b1d into openedx:master Dec 16, 2025
2 of 4 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.

openedx/ccx-keys

2 participants