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.

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

Add ai_aside to code owner mapping #652

Closed
timmc-edx opened this issue May 21, 2024 · 2 comments
Closed

Add ai_aside to code owner mapping #652

timmc-edx opened this issue May 21, 2024 · 2 comments
Labels

Comments

@timmc-edx
Copy link
Member

timmc-edx commented May 21, 2024

On 2024-05-21 we saw some successful transactions with a missing code_owner_squad. These had code_owner_module = ai_aside.config_api.views. This repo is already in the ownership spreadsheet but the module->repo mapping isn't present in... wherever it needs to be.

@jristau1984
Copy link

@timmc-edx - Reading up on code_owner, it sounds like there is a django setting that needs to be applied in the ai_aside IDA... should this be transferred to Cosmonauts to make that update?

@timmc-edx
Copy link
Member Author

Not a Django setting, but an update to either the ownership spreadsheet or the mapping script itself. It would only apply to things in LMS, either dependencies or plugin apps. I actually can't tell which of those without seeing an example of what was happening -- but given that there are no instances of code_owner_module:*aside* in the past month, I'm going to say we can just close this out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done - Long Term Storage
Development

No branches or pull requests

2 participants