Skip to content

fix: format cli.py and test_init.py to pass ruff#38

Merged
Nelson Spence (Fieldnote-Echo) merged 1 commit intomainfrom
fix/lint-format
Mar 12, 2026
Merged

fix: format cli.py and test_init.py to pass ruff#38
Nelson Spence (Fieldnote-Echo) merged 1 commit intomainfrom
fix/lint-format

Conversation

@Fieldnote-Echo
Copy link
Copy Markdown
Member

Summary

  • Runs ruff format on 2 files that were merged with formatting issues
  • Fixes lint CI failure on main

Test plan

  • lint check passes

Copilot AI review requested due to automatic review settings March 11, 2026 23:44
Copy link
Copy Markdown
Collaborator

@project-navi-bot project-navi-bot left a comment

Choose a reason for hiding this comment

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

Auto-approved by navi-bot.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR applies ruff format to previously misformatted Python files to restore lint/format CI green on main.

Changes:

  • Reformat click.option decorators in the CLI to Ruff’s preferred multi-line style.
  • Normalize a small test snippet to Ruff-formatted style in tests/test_init.py.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
tests/test_init.py Applies Ruff formatting to a test helper snippet and surrounding spacing.
src/navi_bootstrap/cli.py Applies Ruff formatting to several @click.option decorators for consistent CLI definition style.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 11, 2026

✅ Grippy Review — PASS

Score: 100/100 | Findings: 0


Commit: 3616b18

@Fieldnote-Echo Nelson Spence (Fieldnote-Echo) merged commit b1cd8c1 into main Mar 12, 2026
12 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.

3 participants