Skip to content

Fix Wagtail transaction names#1705

Draft
hmstepanek wants to merge 1 commit intomainfrom
fix-wagtail-transaction-names
Draft

Fix Wagtail transaction names#1705
hmstepanek wants to merge 1 commit intomainfrom
fix-wagtail-transaction-names

Conversation

@hmstepanek
Copy link
Copy Markdown
Contributor

Overview

Fix Wagtail transaction names.

Related Github Issue

Fixes #1573.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 6, 2026

MegaLinter analysis: Error

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ ACTION actionlint 8 0 0 1.02s
✅ MARKDOWN markdownlint 7 0 0 0 1.3s
❌ PYTHON ruff 1030 1 1 0 1.17s
✅ PYTHON ruff-format 1030 1 0 0 0.38s
✅ YAML prettier 19 0 0 0 1.66s
✅ YAML v8r 19 0 0 5.2s
✅ YAML yamllint 19 0 0 0.75s

Detailed Issues

❌ PYTHON / ruff - 1 error
::error title=ruff (F821),file=newrelic/hooks/framework_wagtail.py,line=30,col=19,endLine=30,endColumn=39::newrelic/hooks/framework_wagtail.py:30:19: F821 Undefined name `is_denied_middleware`
::error title=ruff (F821),file=newrelic/hooks/framework_wagtail.py,line=36,col=20,endLine=36,endColumn=56::newrelic/hooks/framework_wagtail.py:36:20: F821 Undefined name `_nr_wrap_converted_middleware_async_`
::error title=ruff (F821),file=newrelic/hooks/framework_wagtail.py,line=37,col=16,endLine=37,endColumn=46::newrelic/hooks/framework_wagtail.py:37:16: F821 Undefined name `_nr_wrap_converted_middleware_`
::error title=ruff (F821),file=newrelic/hooks/framework_wagtail.py,line=45,col=44,endLine=45,endColumn=74::newrelic/hooks/framework_wagtail.py:45:44: F821 Undefined name `wrap_handle_uncaught_exception`

See detailed reports in MegaLinter artifacts

MegaLinter is graciously provided by OX Security
Show us your support by starring ⭐ the repository

@hmstepanek hmstepanek force-pushed the fix-wagtail-transaction-names branch from a284b4d to 9fd7f38 Compare April 6, 2026 23:27
@mergify mergify bot added the tests-failing Tests failing in CI. label Apr 6, 2026
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 13.33333% with 39 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.76%. Comparing base (1e0a131) to head (9fd7f38).

Files with missing lines Patch % Lines
newrelic/hooks/framework_wagtail.py 0.00% 36 Missing ⚠️
newrelic/hooks/framework_django.py 66.66% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1705      +/-   ##
==========================================
- Coverage   81.96%   81.76%   -0.20%     
==========================================
  Files         214      215       +1     
  Lines       25803    25847      +44     
  Branches     4082     4087       +5     
==========================================
- Hits        21150    21135      -15     
- Misses       3260     3311      +51     
- Partials     1393     1401       +8     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests-failing Tests failing in CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Transaction names are inconsistent using Wagtail

2 participants