Repository navigation
chore: 2U identity cleanup and remove openedx community workflows - #5
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR performs repository identity cleanup as part of ENT-12194 by updating references from openedx/edx-enterprise-data to edx/2u-enterprise-data and removing community-contribution GitHub Actions workflows that aren’t relevant for a 2U-owned repo.
Changes:
- Add an explicit 2U-ownership note and update local editable install path in
README.md. - Update Backstage component name in
catalog-info.yamlto2u-enterprise-data. - Remove openedx community-management workflows from
.github/workflows/and refresh links/release checklist in.github/pull_request_template.md.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| README.md | Adds 2U ownership notice; updates editable install path to ./src/2u-enterprise-data. |
| catalog-info.yaml | Renames Backstage component to match the repo name. |
| .github/workflows/self-assign-issue.yml | Removes openedx community workflow reference. |
| .github/workflows/commitlint.yml | Removes openedx community workflow reference. |
| .github/workflows/add-remove-label-on-comment.yml | Removes openedx community workflow reference. |
| .github/workflows/add-depr-ticket-to-depr-board.yml | Removes openedx community workflow reference. |
| .github/pull_request_template.md | Updates links/checklist to point at edx/2u-enterprise-data and GHA-based publish flow. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Merged
6 tasks done
pbitla-sonata
approved these changes
Aug 18, 2026
sjasti-sonata-svg
approved these changes
Aug 19, 2026
Removing this workflow left the required "Lint Commit Messages" status check with nothing to report, blocking PR merges. Restore it while still dropping the other openedx-specific automation.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.
Suppressed comments (2)
.github/pull_request_template.md:23
- The template mixes
PyPiandPyPI. The canonical spelling isPyPI, and using consistent casing improves clarity (and matches the linked site name).
- [ ] After the [PyPi publish workflow](https://github.com/edx/2u-enterprise-data/actions/workflows/pypi-publish.yml) finishes, verify the version has been pushed to [PyPI](https://pypi.org/project/2u-enterprise-data/)
- [ ] PR created in [edx-analytics-data-api](https://github.com/openedx/edx-analytics-data-api) to upgrade dependencies (including 2u-enterprise-data)
- This **must** be done after the version is visible in PyPi as `make upgrade` in edx-analytics-data-api will look for the latest version in PyPi.
- Note: the 2u-enterprise-data constraint in edx-analytics-data-api **must** also be bumped to the latest version in PyPi.
.github/pull_request_template.md:14
- PR description says 4 community workflows were removed, including
commitlint.yml, but.github/workflows/commitlint.ymlis still present and still referencesopenedx/.githubreusable workflows. Either remove that workflow as well (to match the stated intent) or update the PR description so expectations match what’s actually changing.
- This should be run from either a venv with all the edx-analytics-data-api requirements installed or if you checked out 2u-enterprise-data into the src directory used by edx-analytics-data-api, you can run this command through an edx-analytics-data-api shell.
- It would be `./manage.py makemigrations` in the shell.
- [ ] [Version](https://github.com/edx/2u-enterprise-data/blob/master/enterprise_data/__init__.py) bumped
- [ ] [Changelog](https://github.com/edx/2u-enterprise-data/blob/master/CHANGELOG.rst) record added
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Part of ENT-12194 (sync
edx/2u-enterprise-datawithopenedx/edx-enterprise-data). Its pure identity/cleanup, no upstream code — opened first and independently so the sync PRs that follow aren't carrying unrelated changes.openedx/edx-enterprise-datareferences incatalog-info.yaml(component name),README.md(pip install path), and.github/pull_request_template.md(version/changelog/release/PyPI links, dead Travis CI reference)README.mdcommitlint.yml,self-assign-issue.yml,add-remove-label-on-comment.yml,add-depr-ticket-to-depr-board.ymlNot included: removing the
openedx-webhooks/openedx-pr-watcher-notifyGitHub webhooks — that's a repo Settings change, not a code change, and needs separate explicit sign-off.Test plan
ci.ymlandpypi-publish.ymlstill trigger normally with the 4 workflows removed