Skip to content

Introducing ruff as linter and formatter (basic setup)#2921

Merged
lzchen merged 10 commits into
open-telemetry:mainfrom
emdneto:ruff
Oct 23, 2024
Merged

Introducing ruff as linter and formatter (basic setup)#2921
lzchen merged 10 commits into
open-telemetry:mainfrom
emdneto:ruff

Conversation

@emdneto

@emdneto emdneto commented Oct 22, 2024

Copy link
Copy Markdown
Member

Description

Same as open-telemetry/opentelemetry-python#4223, but added ruff to eachdist.py utility script as well in lintand format commands.

Closes #2449

Signed-off-by: emdneto <[email protected]>
Signed-off-by: emdneto <[email protected]>
Signed-off-by: emdneto <[email protected]>
@emdneto emdneto added build & infra Skip Changelog PRs that do not require a CHANGELOG.md entry labels Oct 22, 2024
@emdneto
emdneto requested a review from a team as a code owner October 22, 2024 13:48
Comment thread scripts/eachdist.py

@emdneto emdneto left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Comment thread pyproject.toml

@xrmx xrmx 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.

This switches opentelemetry from third to first party but LGTM

@lzchen
lzchen merged commit 0adc58b into open-telemetry:main Oct 23, 2024
xrmx pushed a commit to xrmx/opentelemetry-python-contrib that referenced this pull request Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Investigate better tools for linting

8 participants