Skip to content

Wrapping up the leftover tasks in xblocks-extra, prepare for the release#27

Merged
farhan merged 20 commits intomainfrom
farhan/winding-down
Apr 27, 2026
Merged

Wrapping up the leftover tasks in xblocks-extra, prepare for the release#27
farhan merged 20 commits intomainfrom
farhan/winding-down

Conversation

@farhan
Copy link
Copy Markdown
Contributor

@farhan farhan commented Apr 16, 2026

Ticket: #16

Reference PR's: while shifting to uv and writing upgrade-python-requirements.yml

openedx/openedx-aspects#340
openedx/xqueue-watcher#122

Tasks done in this PR:

  • License should be created/moved at the root
  • License has been added copied from xblocks-contrib repo
  • License should be bundled in the packaging
  • Implement python upgrade workflow, Move to uv (from pip)
  • catalog-info.yaml should be created at the root
  • PACKAGE DATA should be revised for all the xblocks as we are not shifting the packaging strategies for example
  • i18n should be supported, translations commands in make file is pending to add
  • Fixing minor issue is left, Irtaza is working on it.
  • Testing of docs is pending
  • Make sure all the readme content of the xblocks has been moved to the xblocks-extra repository. We should create different read-me files for different blocks in their directories for isolation and easy maintenance.
  • We should reference the inter block readmes to the main readme file and list the xblocks repo will contain.
  • We should also make sure all the migrated XBlocks have been marked deprecated and archived before closing this epic.
  • What about the issues created in the main repositories, should we move them all, I think so.
  • What about the un-merged PR's, I think as soon as we make xblocks-extra ready, publish it on pypi, we can drop message on the PR's to shift their PR's.
  • Adding @kdmccormick comment on xblocks-extra:
We should make sure:

Existing sites will be able to uninstall the old packages and install xblocks-extra seamlessly.
There will be documentation explaining that to them.
Both the old repos and xblocks-extra work in Verawood, so that they have a full release to transition.
  • Write transition/migration notes for the Verawood open edx release
  • Mention about archived XBlocks in the Verawood release notes
    Notes have been added here

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.80%. Comparing base (2f72077) to head (7956a76).
⚠️ Report is 1 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (2f72077) and HEAD (7956a76). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (2f72077) HEAD (7956a76)
unittests 2 1
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
- Coverage   86.98%   79.80%   -7.19%     
==========================================
  Files           7       41      +34     
  Lines         269     1139     +870     
  Branches       23       75      +52     
==========================================
+ Hits          234      909     +675     
- Misses         31      203     +172     
- Partials        4       27      +23     
Flag Coverage Δ
unittests 79.80% <ø> (-7.19%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@farhan farhan marked this pull request as draft April 16, 2026 14:08
@farhan farhan force-pushed the farhan/winding-down branch 2 times, most recently from 011d4b8 to 5e1d552 Compare April 17, 2026 07:08
@farhan farhan force-pushed the farhan/winding-down branch from 5e1d552 to a67c3e0 Compare April 17, 2026 07:10
@farhan farhan marked this pull request as ready for review April 21, 2026 15:21
@farhan farhan requested review from feanil and kdmccormick April 21, 2026 15:34
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/update-dependencies.yml Outdated
Comment thread .github/workflows/update-dependencies.yml Outdated
@farhan farhan force-pushed the farhan/winding-down branch from 858d68f to 2d1e85c Compare April 22, 2026 10:04
Comment thread .github/workflows/ci.yml Outdated
Copy link
Copy Markdown
Member

@irtazaakram irtazaakram left a comment

Choose a reason for hiding this comment

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

Make sure to update all the GitHub actions to latest version:
actions/download-artifact@v8
actions/upload-artifact@v7
actions/checkout@v6
codecov/codecov-action@v6

@irtazaakram
Copy link
Copy Markdown
Member

@farhan
Copy link
Copy Markdown
Contributor Author

farhan commented Apr 23, 2026

@irtazaakram

  • workflows added

Make sure to update all the GitHub actions to latest version: actions/download-artifact@v8 actions/upload-artifact@v7 actions/checkout@v6 codecov/codecov-action@v6

  • updated

@farhan farhan merged commit b34de7e into main Apr 27, 2026
6 of 7 checks passed
@farhan farhan deleted the farhan/winding-down branch April 27, 2026 07:11
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.

2 participants