Skip to content

ci: integration tests example regression - #5194

Open
doc-han wants to merge 1 commit into
mainfrom
demo-error-type-allowlist
Open

doc-han wants to merge 1 commit into
mainfrom
demo-error-type-allowlist

Conversation

@doc-han

@doc-han doc-han commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Description

This PR [adds/changes/fixes]... (A description of your work goes here.)

Closes #__

Validation steps

  1. (How can a reviewer validate your work?)

Additional notes for the reviewer

  1. (Is there anything else the reviewer should know or look out for?)

AI Usage

Please disclose whether you've used AI anywhere in this PR (it's cool, we just
want to know!):

  • I have used Claude Code
  • I have used another model
  • I have not used AI

You can read more details in our
Responsible AI Policy

Pre-submission checklist

  • I have performed an AI review of my code (we recommend using /review
    with Claude Code)
  • I have implemented and tested all related authorization policies.
    (e.g., :owner, :admin, :editor, :viewer)
  • I have updated the changelog.
  • I have ticked a box in "AI usage" in this PR

@github-project-automation github-project-automation Bot moved this to New Issues in Core Sep 17, 2026
@github-actions

Copy link
Copy Markdown

Security Review ✅

  • S0 (project scoping): N/A — change is a validate_inclusion on error_type in CompleteRun.new/1 (lib/lightning/runs/handlers.ex:134); no new queries or scoping surfaces touched, and the run's project_id remains server-derived at line 98.
  • S1 (authorization): N/A — no new web-layer entrypoints, controller actions, or handle_events; worker authentication path is unchanged.
  • S2 (audit trail): N/A — no writes to config resources (workflows, credentials, project settings, etc.); the change only tightens input validation on run-completion payloads.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: New Issues

Development

Successfully merging this pull request may close these issues.

1 participant