Skip to content

Format main with ruff #1505

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 19, 2025
Merged

Format main with ruff #1505

merged 1 commit into from
Sep 19, 2025

Conversation

TimPansino
Copy link
Contributor

Overview

  • Format main with ruff to fix linting issues.
  • Add version.py and _version.py files to gitignore.

@TimPansino TimPansino requested a review from a team as a code owner September 19, 2025 18:14
@github-actions GitHub Actions
Copy link

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ ACTION actionlint 6 0 0 0.72s
✅ MARKDOWN markdownlint 7 0 0 0 1.21s
✅ MARKDOWN markdown-link-check 7 0 0 23.98s
✅ PYTHON ruff 945 0 0 0 0.93s
✅ PYTHON ruff-format 945 0 0 0 0.33s
✅ YAML prettier 13 0 0 0 1.42s
✅ YAML v8r 13 0 0 6.91s
✅ YAML yamllint 13 0 0 0.61s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@TimPansino TimPansino enabled auto-merge (squash) September 19, 2025 18:24
@mergify mergify bot added the tests-failing Tests failing in CI. label Sep 19, 2025
@codecov-commenter
Copy link

codecov-commenter commented Sep 19, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 81.40%. Comparing base (5c62b43) to head (d96cd63).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
newrelic/hooks/external_pyzeebe.py 75.00% 1 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #1505      +/-   ##
==========================================
- Coverage   81.43%   81.40%   -0.03%     
==========================================
  Files         211      211              
  Lines       23950    23950              
  Branches     3782     3782              
==========================================
- Hits        19504    19497       -7     
- Misses       3172     3177       +5     
- Partials     1274     1276       +2

☔ 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.

@TimPansino TimPansino merged commit ec3618a into main Sep 19, 2025
107 of 109 checks passed
@TimPansino TimPansino deleted the fix-linting branch September 19, 2025 18:40
@mergify mergify bot removed the tests-failing Tests failing in CI. label Sep 19, 2025
TimPansino added a commit that referenced this pull request Sep 19, 2025
@lrafeei @TimPansino
@dependabot @mergify @umaannamalai @khpeet
Merge main into develop-11.0.0 (#1503)
15b6948 
* Enable coverage for motor instrumentation (#1487)

* Drop CI Image Build Caching (#1493)

* Add deprecation warning for modules (#1490)

* Add deprecation warning for modules

* Megalinter fixes

* Bump github/codeql-action in the github_actions group (#1492)

Bumps the github_actions group with 1 update: [github/codeql-action](https://github.com/github/codeql-action).


Updates `github/codeql-action` from 3.30.1 to 3.30.3
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@f1f6e5f...192325c)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: 3.30.3
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: github_actions
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Add Additional Delay to Test Startup Fixture (#1494)

* Add additional delay to test startup fixture

* Fix assertion error message

---------

Co-authored-by: Uma Annamalai <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Temporarily remove pypy310 test from pymysql and aiomysql (#1501)

* Remove pymysql pypy310 test

* Remove aiomysql pypy310 test

* Falcon sanic testing fix (#1495)

* Remove unsupported Sanic from tests

* Remove unsupported Falcon from tests

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Camunda Pyzeebe Instrumentation (#1385)

* add initial pyzeebe instrumentation/tests

* second attempt at tests (failing)

* updated tests - 1/2 passing locally

* combine function_trace client tests

* fix: py agent team feedback #1

* fix: pyzeebe tests

* chore: ruff formatting

* fix: review updates #2

* fix: next round of updates

* fix: more updates based on feedback

* fix: resource parameter capture

* chore: ruff lint fixes

* fix: no txn tests;add resourceCount attr back

---------

Co-authored-by: Uma Annamalai <[email protected]>

* Format with ruff (#1505)

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Timothy Pansino <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Uma Annamalai <[email protected]>
Co-authored-by: Keagan Peet <[email protected]>
@TimPansino TimPansino added this to the v11.0.0 milestone Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lrafeei lrafeei lrafeei approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v11.0.0
Development

Successfully merging this pull request may close these issues.
3 participants
@TimPansino @codecov-commenter @lrafeei