openedx / xblock-image-explorer

GNU Affero General Public License v3.0
9 stars 39 forks source link

refactor: repo maintenance #109

Closed Agrendalath closed 8 months ago

Agrendalath commented 8 months ago

This:

  1. Adds catalog-info.yaml.
  2. Updates .gitignore.
  3. Removes some leftover code from Python 2.
  4. Modernizes building the package.
  5. Modernizes running test.
  6. Removes unused integration tests.
  7. Updates requirements.
  8. Adds automatic Python requirement upgrades.
  9. Adds the CI workflow to upload the package to PyPI.
  10. Improves CI scripts.
openedx-webhooks commented 8 months ago

Thanks for the pull request, @Agrendalath! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

Agrendalath commented 8 months ago

@feanil, would you mind reviewing this? I tried to make the git history easy to follow.

feanil commented 8 months ago

Yes, I'll take a look tomorrow.

openedx-webhooks commented 8 months ago

@Agrendalath 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.

Agrendalath commented 8 months ago

@feanil, the CI fails to create a git tree in this repo (and in the openedx/xblock-google-drive). Are some additional permissions required for the requirements bot?

feanil commented 7 months ago

Yes, there was a team that needed to be added, I've done so and verified that it is fixed.

https://github.com/openedx/xblock-image-explorer/actions/runs/7631295800/job/20788929574