elastic / elastic/docs-builder

Preview build strategy adjustments

Open
#2,891 2 comments 0 reactions 1 assignee Claimed by @cotti View on GitHub
ai-triaged ai:eng-question automation needs triage stale
Dominant language
C#
Stars
24
Forks
44
Avg merge
1d 7h
Merged PRs (30d)
146

Description

For our short-term hardening goals, we should do away with `pull_request_target` as the trigger for preview builds. `pull_request` should work for the general case, but we should confirm.

It will impact fork PRs, which won't be able to be built anymore under the current method. Since most repos in the org require a successful docs preview action run in their PR workflows, in case it's dealing with a fork PR we need to skip preview generation (but it should still be validated), and return green.

These changes need to be applied to the Vale steps as well.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.