Skip to content

Running tests using python 3.12 - #163

Merged
salman2013 merged 25 commits into
openedx:mainfrom
salman2013:salman/update-python-version
Mar 4, 2024
Merged

Running tests using python 3.12 #163
salman2013 merged 25 commits into
openedx:mainfrom
salman2013:salman/update-python-version

Conversation

@salman2013

@salman2013 salman2013 commented Feb 20, 2024

Copy link
Copy Markdown
Contributor

Description:

Running tests for python 3.11 and 3.12

Ticket: Aximprovements Team (view),
openedx/axim-engineering#1045

@salman2013 salman2013 changed the title chore: update python version support to 3.11 and 3.12 for tests Running tests using python 3.11 and 3.12 Feb 22, 2024
@salman2013
salman2013 marked this pull request as ready for review February 22, 2024 12:01
@salman2013 salman2013 changed the title Running tests using python 3.11 and 3.12 Running tests using python 3.12 Feb 22, 2024
Comment thread requirements/dev.txt Outdated
assert isinstance(package.created, datetime)
assert package.created == package.updated
assert package.created.tzinfo == timezone.utc
# assert package.created.tzinfo == timezone.utc

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.

We shouldn't comment it like this
Either fix it or remove it

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.

Making sure that timestamps are written in UTC is actually important for this app, so I'd really like to understand why this breaks.

Comment thread tox.ini Outdated
Comment thread requirements/test.txt Outdated
# celery
code-annotations==1.6.0
# via -r requirements/test.in
<<<<<<< HEAD

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.

Conflict is not resolved here

Comment thread requirements/quality.txt Outdated
@salman2013
salman2013 requested a review from farhan March 4, 2024 10:25
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