Skip to content

doc inline test#311

Closed
jth-nw wants to merge 2 commits intodevfrom
doc-inline-test
Closed

doc inline test#311
jth-nw wants to merge 2 commits intodevfrom
doc-inline-test

Conversation

@jth-nw
Copy link
Collaborator

@jth-nw jth-nw commented Feb 24, 2026

No description provided.

@jth-nw jth-nw requested a review from a team as a code owner February 24, 2026 19:46
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Documentation Review

No summary was generated.

There are two ways to apply fixes:

  • View them in the comments and apply them individually or in a batch. This only applies to changes made to the file.
  • Reply with @claude here, followed by your instructions (e.g. @claude fix all issues or @claude fix only the spelling errors or @claude fix all other existing issues). You can use this option to fix preexisting issues.

Note: Automated fixes are only available for branches in this repository, not forks.

…lways

- Switch get_pr_diff_valid_lines from gh pr diff to git diff BASE_SHA HEAD_SHA
  --unified=3, ensuring valid line numbers always match the commit_id used in
  the PR review payload and eliminating line: null on inline suggestions
- Add BASE_SHA env var to the Python step
- Update Claude prompt to always write /tmp/review-summary.md even when no
  issues are found, preventing "No summary was generated." on clean documents

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>
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