Skip to content

Fix Wagtail transaction names#1705

Draft
hmstepanek wants to merge 3 commits into
mainfrom
fix-wagtail-transaction-names
Draft

Fix Wagtail transaction names#1705
hmstepanek wants to merge 3 commits into
mainfrom
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 0.9s
✅ MARKDOWN markdownlint 7 0 0 0 1.26s
❌ PYTHON ruff 1050 3 1 0 1.09s
⚠️ PYTHON ruff-format 1050 5 1 0 0.42s
✅ YAML prettier 20 0 0 0 1.66s
✅ YAML v8r 20 0 0 6.15s
✅ YAML yamllint 20 0 0 0.71s

Detailed Issues

❌ PYTHON / ruff - 1 error
::error title=ruff (F821),file=newrelic/hooks/framework_wagtail.py,line=29,col=9,endLine=29,endColumn=14::newrelic/hooks/framework_wagtail.py:29:9: F821 Undefined name `trans`
::error title=ruff (RUF005),file=tests/framework_wagtail/dummy_app/models.py,line=24,col=22,endLine=24,endColumn=52::tests/framework_wagtail/dummy_app/models.py:24:22: RUF005 Consider `[*Page.content_panels, "body"]` instead of concatenation%0A  help: Replace with `[*Page.content_panels, "body"]`
::error title=ruff (RUF005),file=tests/framework_wagtail/dummy_app/models.py,line=30,col=22,endLine=30,endColumn=52::tests/framework_wagtail/dummy_app/models.py:30:22: RUF005 Consider `[*Page.content_panels, "body"]` instead of concatenation%0A  help: Replace with `[*Page.content_panels, "body"]`
::error title=ruff (PTH118),file=tests/framework_wagtail/settings.py,line=68,col=17,endLine=68,endColumn=29::tests/framework_wagtail/settings.py:68:17: PTH118 `os.path.join()` should be replaced by `Path` with `/` operator
::error title=ruff (invalid-syntax),file=tests/framework_wagtail/test_application.py,line=46,col=1,endLine=46,endColumn=4::tests/framework_wagtail/test_application.py:46:1: invalid-syntax: Expected an indented block after function definition
::error title=ruff (RUF005),file=tests/framework_wagtail/urls.py,line=26,endLine=34::tests/framework_wagtail/urls.py:26:15: RUF005 Consider `[*urlpatterns, path("", include(wagtail_urls))]` instead of concatenation%0A  help: Replace with `[*urlpatterns, path("", include(wagtail_urls))]`
⚠️ PYTHON / ruff-format - 1 error
error: Failed to parse tests/framework_wagtail/test_application.py:46:1: Expected an indented block after function definition
4 files reformatted, 1041 files left unchanged

Notices

📣 MegaLinter 9.5.0 is out! Discover the new features and security recommendations in the release announcement. (Skip this info by defining SECURITY_SUGGESTIONS: false)

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-commenter commented Apr 6, 2026

Codecov Report

❌ Patch coverage is 25.92593% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.02%. Comparing base (1e0a131) to head (634c9a5).
⚠️ Report is 62 commits behind head on main.

Files with missing lines Patch % Lines
newrelic/hooks/framework_wagtail.py 0.00% 17 Missing ⚠️
newrelic/hooks/framework_django.py 66.66% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1705      +/-   ##
==========================================
+ Coverage   81.99%   82.02%   +0.03%     
==========================================
  Files         214      216       +2     
  Lines       25803    26350     +547     
  Branches     4082     4158      +76     
==========================================
+ Hits        21156    21613     +457     
- Misses       3254     3320      +66     
- Partials     1393     1417      +24     

☔ 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