build!: Rename the PyPI project to openedx-core#471
Open
kdmccormick wants to merge 2 commits intomainfrom
Open
build!: Rename the PyPI project to openedx-core#471kdmccormick wants to merge 2 commits intomainfrom
openedx-core#471kdmccormick wants to merge 2 commits intomainfrom
Conversation
kdmccormick
commented
Jan 30, 2026
kdmccormick
commented
Jan 30, 2026
ceb984f to
af3b009
Compare
kdmccormick
commented
Feb 17, 2026
kdmccormick
commented
Feb 17, 2026
Cleaning up missed occurances of oepnedx_content.api being imported as "authoring_api".
379f762 to
0e3c749
Compare
openedx-coreopenedx-core
kdmccormick
commented
Feb 17, 2026
e543424 to
dc4c785
Compare
To represent the increasing scope of this repository, we are renaming it from openedx-learning to openedx-core. Practically, this commit: * Updates GitHub hyperlinks in this repo from openedx/openedx-learning -> openedx/openedx-core * Publishes a new PyPI project called openedx-core, version starting at 0.34 * Publishes a final PyPI release of openedx-learning, versioned 0.33, which is empty and just installs openedx-core as a dependency * Updates "Learning Core" to "Open edX Core" in various docs and comments * Updates various docs and comments to represent the project's new scope. BREAKING CHANGE: openedx-learning==0.33.0 will be the final release of openedx-learning. It will transitively install openedx-core==0.34.0 (fully code-compatible) and not receive updates going forward. In order to receive updates, operators should install openedx-core going forward. Part of: #470
dc4c785 to
9ae2c1e
Compare
kdmccormick
commented
Feb 17, 2026
| Context | ||
| ------- | ||
|
|
||
| When this repo was created, it was intended to only encompass learning concepts. The content models were supposed to be read-optimized LMS-friendly representations of learning content. It was imaged that the authoring models would live elsewhere (e.g. Blockstore). That's why this was called "Learning Core". |
Member
Author
There was a problem hiding this comment.
Suggested change
| When this repo was created, it was intended to only encompass learning concepts. The content models were supposed to be read-optimized LMS-friendly representations of learning content. It was imaged that the authoring models would live elsewhere (e.g. Blockstore). That's why this was called "Learning Core". | |
| When this repo was created, it was intended to only encompass learning concepts. The content models were supposed to be read-optimized LMS-friendly representations of learning content. It was imagined that the authoring models would live elsewhere (e.g. Blockstore). That's why this was called "Learning Core". |
kdmccormick
commented
Feb 17, 2026
|
|
||
| * A combined top level Python package with nested apps: ``openedx_core.content.api``, ``openedx_core.cbe.api``, etc. | ||
|
|
||
| * This would make it harder to split apps out into separate repos, later because it would involve either (a) updating all the import statements or (b) having the ``openedx_core`` namespace split across multiple repos. We'd like to remain flexible with code reorganization given the fast-moving and experimental nature of these projects. |
Member
Author
There was a problem hiding this comment.
Suggested change
| * This would make it harder to split apps out into separate repos, later because it would involve either (a) updating all the import statements or (b) having the ``openedx_core`` namespace split across multiple repos. We'd like to remain flexible with code reorganization given the fast-moving and experimental nature of these projects. | |
| * This would make it harder to split apps out into separate repos later, because it would involve either (a) updating all the import statements or (b) having the ``openedx_core`` namespace split across multiple repos. We'd like to remain flexible with code reorganization given the fast-moving and experimental nature of these projects. |
bradenmacdonald
approved these changes
Feb 18, 2026
Contributor
bradenmacdonald
left a comment
There was a problem hiding this comment.
OK, looks good and the plan sounds good to me! I can't really test this but I looked over the diff and didn't see any issues.
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.
Description
To represent the increasing scope of this repository, we are renaming it from openedx-learning to openedx-core. See the ADR for more details: https://github.com/openedx/openedx-core/blob/kdmccormick/openedx-core/docs/decisions/0021-openedx-core.rst
Practically, this PR:
BREAKING CHANGE notice for operators: openedx-learning==0.33.0 will be the final release of openedx-learning. It will transitively install openedx-core==0.34.0 (fully code-compatible) and not receive updates going forward. In order to receive updates, operators should install openedx-core instead.
Notes for reviewers
Supporting info / merge considerations / testing
Before merging this, we'll:
openedx_contentapp #454openedx_contentis a top-level package #468After merging this, I'll:
This is all part of:
openedx-core#470