Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Handle deprecation warnings and drop support for EOL python and django versions #249

Merged
merged 3 commits into from
Aug 15, 2022

Conversation

abhiabhi94
Copy link
Collaborator

  • since python 3.6 and django 2.1 had long reached
    end of life(EOL), we can safely remove support for
    them.

  • also run tests that throw deprecation warnings as errors,
    so that these are caught earlier.

@abhiabhi94
Copy link
Collaborator Author

Hey @Radi85, as of now, I have sent the changes in three separate commits for different parts in this commit, let me know if you want me to make them in separate pull-requests.
The build failures are unrelated.

@codecov-commenter
Copy link

codecov-commenter commented Jul 17, 2022

Codecov Report

Merging #249 (b3a7849) into develop (a98bcad) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #249   +/-   ##
========================================
  Coverage    99.89%   99.89%           
========================================
  Files           56       56           
  Lines         1989     1994    +5     
  Branches       288      289    +1     
========================================
+ Hits          1987     1992    +5     
  Misses           1        1           
  Partials         1        1           
Impacted Files Coverage Δ
comment/__init__.py 100.00% <100.00%> (ø)
comment/conf/__init__.py 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@abhiabhi94 abhiabhi94 changed the title [chore] Handle deprecation warnings and drop support for EOL python and django versions chore: Handle deprecation warnings and drop support for EOL python and django versions Jul 24, 2022
@Radi85
Copy link
Owner

Radi85 commented Aug 14, 2022

Hey @abhiabhi94,
Can you rebase this PR?

@abhiabhi94
Copy link
Collaborator Author

Can you rebase this PR

updated, 👯 GitHub allows people with write access to the repository to update the branch from the UI.

- since python 3.6 and django 2.1 had long reached
  end of life(EOL), we can safely remove support for
  them.
- also run tests that throw deprecation warnings as errors,
  so that these are caught earlier.
@abhiabhi94
Copy link
Collaborator Author

Radico, you probably will need to remove the python 3.6 check from the CI pipeline.

@abhiabhi94
Copy link
Collaborator Author

Hey Radico, thanks for taking the time out for reviewing this.

@abhiabhi94 abhiabhi94 merged commit 0b51944 into Radi85:develop Aug 15, 2022
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