Skip to content

Conversation

@unmultimedio
Copy link
Member

No description provided.

@github-actions
Copy link
Contributor

The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedAug 11, 2025, 9:11 PM

Comment on lines +1037 to +1041
string policy_id = 2;
// policy_ref is the reference of the policy that was enforced.
string policy_ref = 3;
// include_targets were the include targets of the policy enforcement.
repeated string include_targets = 4;
Copy link
Contributor

Choose a reason for hiding this comment

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

For other archive/delete we don't include the full set of fields, just the changeset. Why is this needed?

Copy link
Member Author

Choose a reason for hiding this comment

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

I included the enforcement content (what's being deleted) to be consistent with plugin delete event here, and with repository delete event here.

For enforcements, I'd rather over-audit than under-audit, since they don't evolve with commits/labels.

@unmultimedio unmultimedio merged commit 8183992 into main Aug 12, 2025
12 checks passed
@unmultimedio unmultimedio deleted the jfigueroa/audit-enforcement branch August 12, 2025 14:42
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.

5 participants