dejima/dejima
1
0
Fork 0
Code Issues 29 Pull Requests Actions Packages Projects Releases Wiki Activity

Keep one copy of the pull_request_target rationale #87

Merged
branderson merged 1 commits from wf-comment into main 2026-08-05 22:49:18 -07:00
Conversation 1 Commits 1 Files Changed 1
-23
dejima-agent commented 2026-08-05 22:07:06 -07:00
Collaborator
Copy Link
Copy Source

Summary generated by LLM agent

Removing the transitional trigger in #86 re-added the explanation next to the copy already above it, so tests.yml opened with the same twenty-five lines twice.

Keeps the fuller of the two — it names the controls a branch would be opting out of ("no Docker socket, deny-by-default egress"), which is what makes the paragraph worth reading rather than just accurate.

No behaviour change; the trigger, jobs and checkout refs are untouched.

**Summary generated by LLM agent** Removing the transitional trigger in #86 re-added the explanation next to the copy already above it, so `tests.yml` opened with the same twenty-five lines twice. Keeps the fuller of the two — it names the controls a branch would be opting out of ("no Docker socket, deny-by-default egress"), which is what makes the paragraph worth reading rather than just accurate. No behaviour change; the trigger, jobs and checkout refs are untouched.
dejima-agent added 1 commit 2026-08-05 22:07:07 -07:00
ci: keep one copy of the pull_request_target rationale
tests / sandbox (proxy) (pull_request_target) Successful in 2m2s
Details
tests / sandbox (providers) (pull_request_target) Successful in 4m16s
Details
tests / runner-env (sandbox) (pull_request_target) Successful in 1m28s
Details
tests / sandbox (remote) (pull_request_target) Successful in 5m2s
Details
tests / runner-env (host-docker) (pull_request_target) Successful in 1m22s
Details
tests / sandbox (static) (pull_request_target) Successful in 3m0s
Details
tests / sandbox (project) (pull_request_target) Successful in 5m52s
Details
tests / sandbox (config) (pull_request_target) Successful in 5m59s
Details
tests / sandbox (audit) (pull_request_target) Successful in 8m14s
Details
tests / host-docker (pull_request_target) Failing after 7m46s
Details
531779106e 
Removing the transitional trigger re-added the explanation next to the copy
already above it, so the file opened with the same twenty-five lines twice.
The fuller of the two is kept - it names the controls a branch would be opting
out of, which is the part that makes the paragraph worth reading.
branderson approved these changes 2026-08-05 22:49:14 -07:00
branderson merged commit 3fbd6ad75f into main 2026-08-05 22:49:18 -07:00
Sign in to join this conversation.
Reviewers
No Reviewers
branderson
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
branderson dejima-agent
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: dejima/dejima#87