Skip to content

Fix pylint 4.0.4#47517

Merged
rads-1996 merged 3 commits into
Azure:mainfrom
rads-1996:fix-pylint
Jun 16, 2026
Merged

Fix pylint 4.0.4#47517
rads-1996 merged 3 commits into
Azure:mainfrom
rads-1996:fix-pylint

Conversation

@rads-1996

Copy link
Copy Markdown
Member

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Copilot AI review requested due to automatic review settings June 15, 2026 23:48
@github-actions github-actions Bot added the Monitor - Exporter Monitor OpenTelemetry Exporter label Jun 15, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the azure-monitor-opentelemetry-exporter package (primarily tests, plus one runtime module) to accommodate new/changed Pylint findings in 4.0.4, mostly by removing unused imports and adding targeted # pylint: disable=... pragmas where needed.

Changes:

  • Adds/adjusts Pylint suppression pragmas for specific warnings (e.g., broad-exception-caught, unused-argument, line-too-long, do-not-use-logging-exception, no-name-in-module).
  • Cleans up unused imports and tweaks a couple of test names/strings to avoid Pylint complaints.
  • Minor formatting change to a logged message string (removing implicit string concatenation).

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
sdk/monitor/azure-monitor-opentelemetry-exporter/tests/trace/test_trace_utils.py Adds targeted Pylint suppression for fixedint import.
sdk/monitor/azure-monitor-opentelemetry-exporter/tests/test_utils.py Removes unused imports and renames one test for clearer intent.
sdk/monitor/azure-monitor-opentelemetry-exporter/tests/test_storage.py Removes a redundant local import and suppresses a line-length warning on a long comment line.
sdk/monitor/azure-monitor-opentelemetry-exporter/tests/test_rate_limiter.py Suppresses broad-exception-caught for a thread-safety test and removes an unused local import.
sdk/monitor/azure-monitor-opentelemetry-exporter/tests/test_gen_ai_processor.py Removes unused constant imports.
sdk/monitor/azure-monitor-opentelemetry-exporter/tests/test_base_exporter.py Removes an unused import in a test block.
sdk/monitor/azure-monitor-opentelemetry-exporter/tests/quickpulse/test_exporter.py Removes an unused client import and adds targeted suppressions for argument/line-length warnings.
sdk/monitor/azure-monitor-opentelemetry-exporter/tests/performance_counters/test_performance_counters.py Rewrites a string literal to avoid implicit concatenation.
sdk/monitor/azure-monitor-opentelemetry-exporter/tests/customer_sdk_stats/test_manager.py Suppresses unused-argument for patched test parameters.
sdk/monitor/azure-monitor-opentelemetry-exporter/azure/monitor/opentelemetry/exporter/_performance_counters/_manager.py Suppresses do-not-use-logging-exception at a module-init logging site.

@rads-1996 rads-1996 force-pushed the fix-pylint branch 4 times, most recently from 47205b1 to 47db81d Compare June 16, 2026 19:21
@rads-1996 rads-1996 merged commit 728afe5 into Azure:main Jun 16, 2026
19 checks passed
@rads-1996 rads-1996 deleted the fix-pylint branch June 16, 2026 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Monitor - Exporter Monitor OpenTelemetry Exporter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants