Skip to content

chore: 2U identity cleanup and remove openedx community workflows - #5

Merged
rgopalrao-sonata-png merged 2 commits into
masterfrom
ent-12194/pr0-cleanup-2u-identity
Aug 19, 2026
Merged

rgopalrao-sonata-png merged 2 commits into
masterfrom
ent-12194/pr0-cleanup-2u-identity

Conversation

@rgopalrao-sonata-png

@rgopalrao-sonata-png rgopalrao-sonata-png commented Aug 17, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Part of ENT-12194 (sync edx/2u-enterprise-data with openedx/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.

  • Fixes stale openedx/edx-enterprise-data references in catalog-info.yaml (component name), README.md (pip install path), and .github/pull_request_template.md (version/changelog/release/PyPI links, dead Travis CI reference)
  • Adds an explicit 2U-ownership note to the top of README.md
  • Removes 4 GHA workflows that only make sense for openedx community-contribution management, now that this repo is 2U-only: commitlint.yml, self-assign-issue.yml, add-remove-label-on-comment.yml, add-depr-ticket-to-depr-board.yml

Not included: removing the openedx-webhooks/openedx-pr-watcher-notify GitHub webhooks — that's a repo Settings change, not a code change, and needs separate explicit sign-off.

Test plan

  • No functional code touched (docs/CI-config only)
  • Confirm ci.yml and pypi-publish.yml still trigger normally with the 4 workflows removed

Copilot AI lite review requested due to automatic review settings August 17, 2026 09:04
@rgopalrao-sonata-png
rgopalrao-sonata-png requested review from a team as code owners August 17, 2026 09:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.yaml to 2u-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.

Comment thread .github/pull_request_template.md
@rgopalrao-sonata-png rgopalrao-sonata-png mentioned this pull request Aug 18, 2026
6 tasks done
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.
Copilot AI review requested due to automatic review settings August 19, 2026 11:00

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 PyPi and PyPI. The canonical spelling is PyPI, 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.yml is still present and still references openedx/.github reusable 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

@rgopalrao-sonata-png
rgopalrao-sonata-png merged commit 5a0325f into master Aug 19, 2026
6 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.

4 participants