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

Django 3.2 support #1064

Merged
merged 8 commits into from Mar 16, 2021
Merged

Django 3.2 support #1064

merged 8 commits into from Mar 16, 2021

Conversation

beniwohli
Copy link
Contributor

Add support for Django 3.2

Related issues

closes #1060

@apmmachine
Copy link
Collaborator

apmmachine commented Mar 10, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #1064 updated

  • Start Time: 2021-03-16T12:48:16.591+0000

  • Duration: 21 min 0 sec

  • Commit: f61f028

Test stats 🧪

Test Results
Failed 0
Passed 8521
Skipped 5929
Total 14450

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 8521
Skipped 5929
Total 14450

Django 3.2 now verifies that the app label is a valid Python identifier name.
@beniwohli beniwohli marked this pull request as ready for review March 10, 2021 16:35
@beniwohli beniwohli requested a review from basepi March 10, 2021 16:36
Copy link
Contributor

@basepi basepi left a comment

Choose a reason for hiding this comment

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

LGTM!

@beniwohli beniwohli merged commit 1ab854d into elastic:master Mar 16, 2021
APM-Agents (OLD) automation moved this from In Progress to Done Mar 16, 2021
@beniwohli beniwohli deleted the django-32 branch March 16, 2021 13:15
basepi added a commit to basepi/apm-agent-python that referenced this pull request Sep 3, 2021
basepi added a commit that referenced this pull request Sep 3, 2021
beniwohli added a commit to beniwohli/apm-agent-python that referenced this pull request Sep 14, 2021
* Use `elasticapm` as Django app label

Django 3.2 now verifies that the app label is a valid Python identifier name.

* use `re_path` instead of `url` if available

this reduces the deprecation warning noise in newer Django releases

* added Django 3.2 to test matrix

* Fix issue when resetting config after new value has been added (elastic#1068)

* add test that demonstrates problem from elastic#1067

* check if key existed in original config before setting up callback for it

* update changelog
beniwohli pushed a commit to beniwohli/apm-agent-python that referenced this pull request Sep 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

Add support for Django 3.2
3 participants