Skip to content

perf: teak release support - #319

Merged
luisfelipec95 merged 17 commits into
masterfrom
lfc/teak-support
Jun 19, 2025
Merged

perf: teak release support#319
luisfelipec95 merged 17 commits into
masterfrom
lfc/teak-support

Conversation

@luisfelipec95

@luisfelipec95 luisfelipec95 commented Jun 10, 2025

Copy link
Copy Markdown
Contributor

Description

This PR adds support to Teak.

  1. Updated the requirements to fit the latest release
  2. Removes tests and constraints for Python 3.8

Testing instructions

  1. Install a brand new environment using Tutor Teak
  2. Install eox-tenant and eox-core using this branch
  3. Run the test cases descibed here

@luisfelipec95
luisfelipec95 requested a review from a team as a code owner June 10, 2025 01:37
@magajh

magajh commented Jun 12, 2025

Copy link
Copy Markdown
Contributor

@luisfelipec95 why are the tests here failing? can we please fix them?

Comment thread CHANGELOG.md Outdated
Comment thread README.rst
Comment thread README.rst Outdated
Comment thread eox_core/edxapp_wrapper/backends/users_l_v1.py Outdated
Comment thread eox_core/edxapp_wrapper/backends/users_q_v1.py Outdated
Comment thread eox_core/api/v1/tests/unit/test_grades.py Outdated
Comment thread .github/workflows/tests.yml
@magajh

magajh commented Jun 13, 2025

Copy link
Copy Markdown
Contributor

@luisfelipec95 Please let us know once this has been fully tested in a remote environment and once you are sure everything is working correctly

magajh
magajh previously approved these changes Jun 19, 2025

@magajh magajh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If we have tested this and made sure all test cases work in a remote env, then LGTM!

@luisfelipec95

Copy link
Copy Markdown
Contributor Author

The dependencies are up to date and the test cases were run in the remote environment.

@magajh
magajh self-requested a review June 19, 2025 16:01

@magajh magajh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@luisfelipec95
luisfelipec95 merged commit 49c26c1 into master Jun 19, 2025
4 of 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.

3 participants