Skip to content

Fix pylint 4.0.4 in the distro#47532

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

Fix pylint 4.0.4 in the distro#47532
rads-1996 merged 2 commits into
Azure:mainfrom
rads-1996:fix-pylint-distro

Conversation

@rads-1996

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings June 16, 2026 19:03
@github-actions github-actions Bot added the Monitor - Distro Monitor OpenTelemetry Distro label Jun 16, 2026
@rads-1996 rads-1996 force-pushed the fix-pylint-distro branch from 4e1cd79 to 04bc745 Compare June 16, 2026 19:04

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 package to address new/updated pylint findings (notably around exception logging and line-length) and refreshes the package’s API documentation artifacts.

Changes:

  • Adjusted tests to remove unused mocks/imports, add targeted pylint suppressions, and reformat long connection strings.
  • Added/expanded pylint suppressions related to exception logging (including module-level disables in the browser SDK loader).
  • Added generated API surface files (api.md, api.metadata.yml).

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 14 comments.

Show a summary per file
File Description
sdk/monitor/azure-monitor-opentelemetry/tests/browserSdkLoader/test_snippet_injector.py Removes unused mock imports and reformats long connection string constants.
sdk/monitor/azure-monitor-opentelemetry/tests/browserSdkLoader/test_integration.py Reformats long connection strings and adds pylint suppressions in one test.
sdk/monitor/azure-monitor-opentelemetry/tests/browserSdkLoader/test_django_middleware.py Removes unused mock imports, reformats long connection string, and simplifies a conditional branch.
sdk/monitor/azure-monitor-opentelemetry/tests/browserSdkLoader/test_config.py Removes unused mock imports and reformats a long connection string constant.
sdk/monitor/azure-monitor-opentelemetry/samples/logging/modify_logs.py Adds a pylint suppression for a long line in a sample.
sdk/monitor/azure-monitor-opentelemetry/azure/monitor/opentelemetry/_diagnostics/diagnostic_logging.py Adds a pylint suppression around exception logging (with some redundancy).
sdk/monitor/azure-monitor-opentelemetry/azure/monitor/opentelemetry/_browser_sdk_loader/snippet_injector.py Adds module-level and inline pylint suppressions for exception logging warnings.
sdk/monitor/azure-monitor-opentelemetry/azure/monitor/opentelemetry/_browser_sdk_loader/django_middleware.py Adds a module-level pylint suppression for exception logging.
sdk/monitor/azure-monitor-opentelemetry/azure/monitor/opentelemetry/_browser_sdk_loader/init.py Adds a pylint suppression on a warning log with exc_info=True.
sdk/monitor/azure-monitor-opentelemetry/api.metadata.yml Adds generated API metadata for the package.
sdk/monitor/azure-monitor-opentelemetry/api.md Adds generated API markdown for the package surface.

@rads-1996 rads-1996 merged commit f072546 into Azure:main Jun 16, 2026
19 checks passed
@rads-1996 rads-1996 deleted the fix-pylint-distro branch June 16, 2026 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Monitor - Distro Monitor OpenTelemetry Distro

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants