Add Flake8 Linting to CI - #12
Conversation
This change splits the linting process in the CI workflow into two separate jobs: Lint (Ruff) and Lint (Flake8). Both jobs must pass before the test jobs are executed. The dependency graph in GitHub Actions will now show these as two parallel blocks. Additionally: - Updated `requirements-dev.txt` to include `flake8`. - Fixed a bug in `shellmind/ai.py` where `_direct_shell_command` was not correctly handling 'shell' and 'run' prefixes, causing test failures. - Ensured CI jobs use `requirements-dev.txt` for consistency. Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
1 participant
I have added Flake8 linting to the CI workflow. The linting is now split into two separate jobs, 'Lint (Ruff)' and 'Lint (Flake8)', which run in parallel and are both required for the subsequent test jobs to start. I also updated 'requirements-dev.txt' and fixed a bug in the command parsing logic that was discovered during testing.
PR created automatically by Jules for task 5710132515424745271 started by @Natuworkguy