-
Notifications
You must be signed in to change notification settings - Fork 25
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
chore: add catalog-info.yml #121
Conversation
Thanks for the pull request, @ichintanjoshi! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
# | ||
# If you need a new team created, create an issue with Axim engineering: | ||
# https://github.com/openedx/axim-engineering/issues/new/choose | ||
owner: group:openedx-unmaintained |
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.
This looks right according to https://github.com/orgs/openedx/teams/openedx-unmaintained:
An empty team for use with Backstage to indicate that a repository or service is unmaintained.
Unless someone has recently volunteered to maintain it :)
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.
We discussed with @cmltaWt0 recently that RG has required expertise to maintain e2e tests repo
We will to finalize discussion tomorrow and come back with the final update if we are ready to maintain this repo on behalf of RG
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.
@sarina I have commented on the maintenance sheet regarding the repository. I would be happy to maintain the repository, and we also have an AQA engineer at RG who is interested in maintaining it.
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.
@cmltaWt0 awesome! I'm going to merge this stub catalog-info, and you/your AQA engineer can update with the maintainer(s) and/or maintainer group when you're ready to take over.
catalog-info.yaml
Outdated
# We use it in Open edX repos to have a comma-separated list of GitHub user | ||
# names that might be interested in changes to the architecture of this | ||
# component. | ||
openedx.org/arch-interest-groups: "feanil" |
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.
I'm not sure @feanil will want this responsibility...
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.
I agree, but that's what it came with in template, and I am not sure it'll allow something like openedx-unmaintained
over there ?
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.
Since it is optional, I think I would make it
openedx.org/arch-interest-groups: ""
but I'd wait for Feanil to chime in on whether he meant to be the default arch interest group by putting himself in the template.
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.
I found reference in other repos @sarina and it can be
openedx.org/arch-interest-groups: ""
So have updated that and linting too
3cd8c76
to
388093e
Compare
# | ||
# If you need a new team created, create an issue with Axim engineering: | ||
# https://github.com/openedx/axim-engineering/issues/new/choose | ||
owner: group:openedx-unmaintained |
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.
@cmltaWt0 awesome! I'm going to merge this stub catalog-info, and you/your AQA engineer can update with the maintainer(s) and/or maintainer group when you're ready to take over.
@ichintanjoshi 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
Description