Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Add Django 4.2 support in CI in Registrar #550

Closed
wants to merge 18 commits into from
Closed

Conversation

irtazaakram
Copy link
Contributor

@irtazaakram irtazaakram commented Aug 28, 2023

@irtazaakram irtazaakram marked this pull request as draft August 28, 2023 08:52
@irtazaakram irtazaakram force-pushed the add-django42 branch 3 times, most recently from e845934 to a50d340 Compare August 29, 2023 07:25
@UsamaSadiq UsamaSadiq marked this pull request as ready for review September 25, 2023 14:40
commands = {posargs:pytest}
setenv =
Copy link
Contributor

Choose a reason for hiding this comment

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

Tox gives error locally due to boto issue.

@awais786 awais786 changed the title [Django 4.2]: Add Django 4.2 in CI in Registrar Add Django 4.2 support in CI in Registrar Oct 6, 2023
@awais786 awais786 mentioned this pull request Oct 6, 2023
4 tasks
@awais786
Copy link
Contributor

awais786 commented Oct 6, 2023

@zacharis278 you can review this again.

@@ -14,6 +14,8 @@

from ..api_client import DISCOVERY_API_TPL, DiscoveryServiceClient
from ..discovery_cache import ProgramDetails
from ..tests.test_api_client import \
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this change needed?

@awais786 awais786 closed this Oct 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Registrar]: Django 4.2 Upgrade
5 participants