Skip to content

Added ruff ci linting workflow - #64

Merged
codepiyusss merged 7 commits into
codepiyusss:mainfrom
gtref:main
Oct 24, 2026
Merged

codepiyusss merged 7 commits into
codepiyusss:mainfrom
gtref:main

Conversation

@gtref

@gtref gtref commented Oct 23, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

This PR adds ruff ci linting for python as a new ci work flow and updates cli.py so that when no args are added to it when it is run, it returns the help menu.

Related issue

Closes #40

Checklist

  • I commented on the issue first and was assigned
  • I removed the xfail marker (if fixing a bug)
  • I added or updated tests
  • pytest passes locally
  • This PR only addresses one issue

gtref added 4 commits October 24, 2026 10:03
… is run with no args the tool returns the argparse provided help menu

Signed-off-by: G-type <162748908+gtref@users.noreply.github.com>
Signed-off-by: G-type <162748908+gtref@users.noreply.github.com>
… main branch was succsessfull or not

Signed-off-by: G-type <162748908+gtref@users.noreply.github.com>
@gtref
gtref marked this pull request as ready for review October 23, 2026 21:24
@gtref

gtref commented Oct 23, 2026

Copy link
Copy Markdown
Contributor Author

@codepiyusss I have completed the ruff ci workflow but there are also some other changes I made.

@codepiyusss

Copy link
Copy Markdown
Owner

The CI check for ruff/lint (pull_request) is failing because Ruff detected formatting/lint issues in this branch. The workflow output shows:

  • “9 files would be reformatted”
  • “14 files already formatted”
  • exit code 1

This means the PR is failing the repository’s formatting/lint gate before merge. Please run the formatter and lint fixes locally, then commit and push the updated files:

  • ruff format .
  • ruff check --fix
    If there are conflicts with main, resolve them first and then rebase or merge before pushing again. The PR will update automatically when you push to the same branch, so there is no need to open a new PR.

Once the formatting/lint issues are fixed, the CI check should pass. Thanks!

@codepiyusss
codepiyusss merged commit 6f40254 into codepiyusss:main Oct 24, 2026
4 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.

Tooling: add ruff linting to CI

2 participants