Skip to content

style(tests): clear the #270 Ruff formatting gate - #1

Open
dinakarjs wants to merge 1 commit into
review/270-basefrom
fix/270-format-gate
Open

dinakarjs wants to merge 1 commit into
review/270-basefrom
fix/270-format-gate

Conversation

@dinakarjs

Copy link
Copy Markdown
Owner

Upstream agentrust-io#270 at 365bed8 fails the CI-required ruff format --check src/ tests/ in four changed test files. This author-adoption companion applies the pinned formatter to those files. All four Python ASTs are identical before and after; no behavior changes.

Adopt commit c3a9dd3 into the existing upstream PR. The base branch is frozen at its reviewed head.

Validation: the original head passed the full unit/conformance suite (879 passed, 11 skipped; 90.53% coverage), Ruff lint, mypy (47 files), and Bandit in a hash-locked editable-install environment. The original formatting gate failed; after this formatting-only change all 107 files pass that gate and Ruff lint passes. Full tests were not rerun after the AST-identical formatting. Hosted CI and independent approval remain separate gates.

Signed off under DCO. Prepared with OpenAI Codex.

Format four changed test files without altering their Python ASTs.

Signed-off-by: Srinivasa Dinakar Jainti <dinakarjs@gmail.com>
@github-actions

github-actions Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

🟡 Contributor Check: MEDIUM

Check Result
Profile MEDIUM
Credential LOW
Overall MEDIUM

Automated check by AgenTrust Contributor Check.

@github-actions github-actions Bot added the needs-review:MEDIUM Contributor check flagged MEDIUM risk label Oct 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review:MEDIUM Contributor check flagged MEDIUM risk

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant