Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: Add Ruff GitHub Workflow #12

Conversation

error9098x
Copy link
Collaborator

@error9098x error9098x commented Jun 22, 2024

Features:

  • Adds a GitHub workflow for Ruff linter for better code quality.
  • Automatically runs after every PR.
  • Automatically tries to fix the issues and raise another PR with fixed issues.
  • If unable to fix the issues, the workflow fails

Screenshot from 2024-06-22 14-42-18

Screenshot from 2024-06-22 14-43-06

@luarss luarss self-requested a review June 22, 2024 15:28
@luarss
Copy link
Collaborator

luarss commented Jun 22, 2024

Can you add some docs for this?

.github/workflows/ruff_linter.yml Outdated Show resolved Hide resolved
.github/workflows/ruff_linter.yml Outdated Show resolved Hide resolved
@luarss luarss linked an issue Jun 22, 2024 that may be closed by this pull request
@luarss luarss requested a review from dralabeing July 6, 2024 10:39
Copy link
Member

@vvbandeira vvbandeira left a comment

Choose a reason for hiding this comment

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

I am just commenting on the GH workflow. In the future, keep PRs single-focused, either CI-related or related to code format or code changes. Also, let me know if GHA VMs meet the required HW resources or when you would like to set up an automatic deployment of the front/backend. This will require some dev time from me, so let me know in advance so I can plan accordingly.

.github/workflows/ruff_linter.yml Outdated Show resolved Hide resolved
@luarss luarss force-pushed the feature/add-ruff-github-workflow branch from 858cce5 to ef4d19b Compare July 7, 2024 02:36
@luarss
Copy link
Collaborator

luarss commented Jul 7, 2024

@vvbandeira Understood, it's just that this is the very first addition of the CI, so it warrants some code/requirement changes.

@luarss luarss requested a review from vvbandeira July 7, 2024 03:08
@vvbandeira vvbandeira mentioned this pull request Jul 8, 2024
vvbandeira
vvbandeira previously approved these changes Jul 8, 2024
Copy link
Member

@vvbandeira vvbandeira left a comment

Choose a reason for hiding this comment

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

LGTM

vvbandeira
vvbandeira previously approved these changes Jul 9, 2024
error9098x and others added 9 commits July 9, 2024 14:06
Signed-off-by: error9098x <[email protected]>
Signed-off-by: Jack Luar <[email protected]>
Signed-off-by: Jack Luar <[email protected]>
Signed-off-by: Jack Luar <[email protected]>
Signed-off-by: Jack Luar <[email protected]>
Signed-off-by: Jack Luar <[email protected]>
Signed-off-by: Song Luar <[email protected]>
Signed-off-by: Song Luar <[email protected]>
@luarss luarss requested a review from vvbandeira July 9, 2024 14:16
@vvbandeira vvbandeira merged commit f64c791 into The-OpenROAD-Project:master Jul 9, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add docs for Ruff usage
3 participants