Skip to content

ci: Add Ruff linter as GitHub Action.#864

Open
CagriYonca wants to merge 2 commits intomainfrom
github_action_ruff
Open

ci: Add Ruff linter as GitHub Action.#864
CagriYonca wants to merge 2 commits intomainfrom
github_action_ruff

Conversation

@CagriYonca
Copy link
Copy Markdown
Contributor

Reopening an old PR closed some time ago.

@pvital - Executes Ruff as a linter tool to check code patterns.
@CagriYonca - Adds information about pre-commit to the CONTRIBUTING.md

Ruff linter has been tested here.

@CagriYonca CagriYonca requested a review from a team as a code owner April 14, 2026 12:15
@CagriYonca CagriYonca force-pushed the github_action_ruff branch 5 times, most recently from db3414e to 6163d09 Compare April 15, 2026 08:58
Copy link
Copy Markdown
Member

@pvital pvital left a comment

Choose a reason for hiding this comment

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

Few requests.

Comment thread .github/workflows/linter.yml
Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md Outdated
Executes Ruff as linter tool to check code pattern.

Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
Signed-off-by: Cagri Yonca <cagri@ibm.com>
@CagriYonca CagriYonca force-pushed the github_action_ruff branch 3 times, most recently from 4e208ea to e0bee6a Compare April 22, 2026 07:50
@CagriYonca CagriYonca requested a review from pvital April 22, 2026 07:59
…G.md

Signed-off-by: Cagri Yonca <cagri@ibm.com>
Co-authored-by: Paulo Vital <paulo.vital@ibm.com>
Signed-off-by: Cagri Yonca <cagri@ibm.com>
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Are these file changes only for testing the GH Action?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's not for testing. The pipeline just ran ruff and found this part to be fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants