Skip to content

Conversation

@dulmandakh
Copy link
Contributor

@dulmandakhdulmandakh commented Aug 14, 2024

Description of the Change

Django versions below 4.2 are no longer supported, so this PR removes mentions and CI for these versions.

Checklist

  • PR only contains one change (considered splitting up PR)
  • unit-test added
  • documentation updated
  • CHANGELOG.md updated (only for user relevant changes)
  • author name in AUTHORS

@dulmandakhdulmandakh changed the title drop support for Django below 4.2drop support for Django versions below 4.2Aug 14, 2024
@codecov
Copy link

codecovbot commented Aug 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.32%. Comparing base (51d9798) to head (75c28f8).
Report is 1 commits behind head on master.

Additional details and impacted files
@@ Coverage Diff @@## master #1455 +/- ## ========================================== - Coverage 97.34% 97.32% -0.03%  ========================================== Files 32 32 Lines 2112 2128 +16 ========================================== + Hits 2056 2071 +15 - Misses 56 57 +1 

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

@dulmandakh
Copy link
ContributorAuthor

@n2ygk please review and merge. I have no idea why codecov is failing. Let's make CI lean and clean.

@n2ygk
Copy link
Contributor

Yeah codecov keeps misreporting that tiny delta but it's unclear why. Luckily it done's block accepting the PR.

@n2ygkn2ygkforce-pushed the remove-django-below-4.2 branch from 75c28f8 to 2e2af76CompareAugust 14, 2024 17:47
@n2ygkn2ygk self-requested a review August 14, 2024 17:50
Copy link
Contributor

@n2ygkn2ygk left a comment

Choose a reason for hiding this comment

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

Nice work! Thanks for simplifying the workfliws.

One question regarding python 3.8, 3.9 needs to be resolved.

@n2ygkn2ygk merged commit 7e13413 into django-oauth:masterAug 15, 2024
@dulmandakhdulmandakh deleted the remove-django-below-4.2 branch August 15, 2024 14:10
@n2ygkn2ygk added this to the 3.0.0 milestone Aug 15, 2024
Sign up for freeto 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

@dulmandakh@n2ygk