openedx / axim-engineering

GitHub Issue repository for the Axim engineering team
https://openedx.atlassian.net/wiki/spaces/COMM/pages/3241640370/Axim+Collaborative+Engineering+Team
4 stars 2 forks source link

[GH Request] Repo for https://github.com/open-craft/xblock-skill-tagging #687

Closed irfanuddinahmad closed 1 year ago

irfanuddinahmad commented 1 year ago

Firm Name

2U

Urgency

Medium (< 2 weeks)

Requested Change

New repo for https://github.com/open-craft/xblock-skill-tagging

Reasoning

Need to push releases to PyPI

openedx-workflow-automation[bot] commented 1 year ago

Thank you for your report! @openedx/tcril-oncall will triage within a business day. Simple requests usually take 2-3 business days to resolve; more complex requests could take longer.

bmtcril commented 1 year ago

Hi @irfanuddinahmad just to confirm, this is a request for tCRIL to transfer the repo into the openedx org? Is this code that OpenCraft owns, or has is it owned by 2U under a development agreement? I only ask because the firm name on the ticket is "2U".

irfanuddinahmad commented 1 year ago

Hi @bmtcril This work is the outcome of a blended project between 2U/edX and OpenCraft.

bmtcril commented 1 year ago

@irfanuddinahmad to transfer ownership of this repo we will need to follow the directions listed here: https://openedx.atlassian.net/wiki/spaces/COMM/pages/3438903337/On-call+Playbooks#%F0%9F%8F%97-Transferring-repositories-into-openedx . Would you all be comfortable giving me admin access so we can transfer instead of forking this?

We will need an affirmative statement from a manager or principal engineer from whichever organization actually owns the code that's being transferred along the lines of:

"I confirm that is intentionally submitting the repo code as a contribution to the Open edX public platform codebase under the existing -tCRIL license and contribution agreements."

@antoviaque usually handles this on the OpenCraft side.

antoviaque commented 1 year ago

@bmtcril Thanks for the ping - yes, I can confirm that OpenCraft is "intentionally submitting the repo code as a contribution to the Open edX public platform codebase under the existing - tCRIL license and contribution agreements." CC @navinkarkera

bmtcril commented 1 year ago

Great, thanks. So I think the next step is to grant me admin on the repo and I'll do the transfer.

antoviaque commented 1 year ago

@bmtcril Done! Don't hesitate if you need anything else.

bmtcril commented 1 year ago

Thanks all, I've done the transfer and will start setting up permissions. If there is anyone who would like / need core contributor access to the repo, feel free to open a thread on the forums and I'll send my approval.

irfanuddinahmad commented 1 year ago

@bmtcril we want to merge https://github.com/openedx/xblock-skill-tagging/pull/9 ... however, we don't seem to have the rights to do so

sameenfatima78 commented 1 year ago

@bmtcril Can @openedx/enterprise-markhors get core contributor access to this repo? We want to publish this repo to PyPI but I can't seem to draft a release right now.

bmtcril commented 1 year ago

@irfanuddinahmad @sameenfatima78 sorry I missed the original question. Are there existing core contributors who we can expand access to this repo for or are you looking to onboard new core contributors with access here?

irfanuddinahmad commented 1 year ago

@irfanuddinahmad @sameenfatima78 sorry I missed the original question. Are there existing core contributors who we can expand access to this repo for or are you looking to onboard new core contributors with access here?

@bmtcril Thanks. Can you add me and Sameen.

bmtcril commented 1 year ago

@irfanuddinahmad @sameenfatima78 so neither of you are actually in the core contributor program, my understanding is that you are working on this project in a 2U capacity under their CLA? I think @antoviaque as a representative of the owning organization would need to designate you as maintainers here since this isn't a historically inherited 2U repo.

Presuming you want to be the maintainers of this project and all that entails. :)

irfanuddinahmad commented 1 year ago

@bmtcril This project is funded by 2U and contracted to OpenCraft. We would be maintaining it under the 2U CLA.

antoviaque commented 1 year ago

@bmtcril Happy to help with this as needed, for the nomination process, or anything related to it.

@irfanuddinahmad @sameenfatima78 Are you looking to become maintainers as per https://docs.openedx.org/projects/openedx-proposals/en/latest/processes/oep-0055-proc-project-maintainers.html#abstract ? If so, I could guide you through the process to become maintainer, if you want?

irfanuddinahmad commented 1 year ago

@antoviaque At the moment we just want to release the initial version to PyPI so that we get unblocked on our work. It would be an honor to maintain this codebase, however, we donot currently have the bandwidth to fulfill all of the maintainer responsibilities.

bmtcril commented 1 year ago

Ok, I don't want to keep you all blocked on this so I can push out a release, but I can't become the maintainer of this repo simply because I pushed the button to bring it into the organization. I won't have the context to make appropriate decisions about it or review changes.

The version is currently at 0.1.0, is the code on main now what you want pushed out or do you want that other PR merged first?

irfanuddinahmad commented 1 year ago

Thanks @bmtcril Yes. main can be released.

bmtcril commented 1 year ago

@irfanuddinahmad it looks like it already was, back in March: https://pypi.org/project/skill-tagging/ the only changes since then have been to Github actions. I think if there's anything more to be done here you should open a new ticket since we're well past the original request.