Skip to content

chore(ci): SHA-pin reusable auto-code-review workflow - #48

Open
Coding-Dev-Tools wants to merge 3 commits into
masterfrom
chore/sha-pin-auto-code-review
Open

chore(ci): SHA-pin reusable auto-code-review workflow#48
Coding-Dev-Tools wants to merge 3 commits into
masterfrom
chore/sha-pin-auto-code-review

Conversation

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner

SHA-pins the org-hosted reusable workflow auto-code-review.yml@main to commit 9114bb00d7f7a8e1a06310700bb7a539c3034c37 for supply-chain immutability.

All other workflow actions in this repo are already SHA-pinned; this was the last unpinned reference.


🤖 Generated by senior-dev rotation

@github-actions

github-actions Bot commented Aug 16, 2026

Copy link
Copy Markdown

🤖 Automated Code Review

✅ Ruff Lint — No issues

⚠️ Ruff Format — Formatting needed

unformatted: File would be reformatted
  --> README.md:28:1
   |
27 | import pandas as pd
   - df = pd.read_csv('huge_file.csv')  # OOM on 5GB+
   - df.to_parquet('output.parquet')
28 +
29 + df = pd.read_csv("huge_file.csv")  # OOM on 5GB+
30 + df.to_parquet("output.parquet")
31 | ```
   |

unformatted: File would be reformatted
 --> conftest.py:2:1
  |
1 | """pytest configuration — add project src to Python path and skip rate limits."""
2 +
3 | import os
  |

unformatted: File would be reformatted
   --> src/datamorph/cli.py:103:15
    |
102 | @click.option("--pattern", default="*", help="File glob pattern (default: all files)")
    - @click.option(
    -     "--recursive", "-r", is_flag=True, help="Search subdirectories recursively"
    - )
103 + @click.option("--recursive", "-r", is_flag=True, help="Search subdirectories recursively")
104 | @click.option("--csv-delimiter", default=",", help="CSV delimiter")
--------------------------------------------------------------------------------

✅ Secret Detection — Clean

✅ Large Files — Within limits

📊 Diff Stats — 1 file(s) changed

 .github/workflows/auto-code-review.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Verdict: ⚠️ Warnings — Lint/format issues found. Recommend fixing before merge.

Automated by Coding-Dev-Tools/.github reusable workflow.

Address ruff format warnings from automated code review bot on PR #48.

Changes: blank line after module docstring in conftest.py, consolidate multi-line click.option decorators in cli.py.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1f711ef38f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/datamorph/cli.py
Comment thread conftest.py
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.

1 participant