Skip to content

fix(pull-requests): match native Gitea warning status semantics - #23

Open
kalvenschraut wants to merge 1 commit into
gitea/adapter-typesfrom
gitea/warning-checks
Open

fix(pull-requests): match native Gitea warning status semantics#23
kalvenschraut wants to merge 1 commit into
gitea/adapter-typesfrom
gitea/warning-checks

Conversation

@kalvenschraut

@kalvenschraut kalvenschraut commented Sep 5, 2026

Copy link
Copy Markdown
Member

A Gitea warning commit status contributes to a failing native check summary, but the PR detail previously showed that check as neutral. Map warning to failure so the list and detail agree with Gitea’s native status combiner.

The 59 focused Gitea API tests, final server typecheck, and targeted lint pass, including a warning-status regression. This adds one case to the integrated stack’s 228 previously passing focused tests.

Model: GPT-6 Astra; native-source audit GPT-5.6 Luna. Harness: Codex.

@kalvenschraut

Copy link
Copy Markdown
Member Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Sep 5, 2026

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@coderabbitai

coderabbitai Bot commented Sep 5, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 2 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 10 included reviews currently available.

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 73f35c73-4b6f-46c8-8acb-f9af20c6593e

📥 Commits

Reviewing files that changed from the base of the PR and between 256fd6f and 93580c0.

📒 Files selected for processing (2)
  • apps/server/src/pullRequest/GiteaPullRequestApi.test.ts
  • apps/server/src/pullRequest/GiteaPullRequestApi.ts

Comment @coderabbitai help to get the list of available commands.

@kalvenschraut
kalvenschraut marked this pull request as ready for review September 5, 2026 05:58
@kalvenschraut

Copy link
Copy Markdown
Member Author

@coderabbitai review

Please review this PR at its exact current head in stack pingdotgg#6. Verify correctness, regressions, security boundaries, and focused coverage against its current base, and report any actionable findings with file references.

@coderabbitai

coderabbitai Bot commented Sep 5, 2026

Copy link
Copy Markdown

@kalvenschraut I will review pull request #23 at its current head against its current base. I will report actionable findings with file references.

⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

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