Skip to content

adding logging format lint check#4525

Merged
xrmx merged 4 commits intoopen-telemetry:mainfrom
thogar-computer:feature/linting-ruff-G
Apr 9, 2025
Merged

adding logging format lint check#4525
xrmx merged 4 commits intoopen-telemetry:mainfrom
thogar-computer:feature/linting-ruff-G

Conversation

@thogar-computer
Copy link
Contributor

@thogar-computer thogar-computer commented Apr 2, 2025 •
edited by xrmx
Loading

Description

Refs #4227

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • test-opentelemetry-exporter-otlp-proto-common

Does This PR Require a Contrib Repo Change?

  • Yes. - Link to PR:
  • No.

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

@thogar-computer thogar-computer requested a review from a team as a code owner April 2, 2025 16:27
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Apr 2, 2025 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

pmcollins
pmcollins approved these changes Apr 7, 2025
View reviewed changes
Copy link
Member

@pmcollins pmcollins left a comment

Choose a reason for hiding this comment

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

LGTM -- thanks. Please take a look at the pre-commit failure.

@xrmx xrmx moved this to Easy to review / merge / close in Python PR digest Apr 9, 2025
@xrmx xrmx added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Apr 9, 2025
@xrmx xrmx merged commit 34b3ac6 into open-telemetry:main Apr 9, 2025
386 checks passed
@github-project-automation github-project-automation bot moved this from Easy to review / merge / close to Done in Python PR digest Apr 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xrmx xrmx xrmx approved these changes

@pmcollins pmcollins pmcollins approved these changes

@emdneto emdneto emdneto approved these changes

Assignees

No one assigned

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

Python PR digest
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@thogar-computer @xrmx @pmcollins @emdneto