Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove author_association check in co-docs-builder.yml #3938

Merged
merged 1 commit into from
May 28, 2024
Merged

Remove author_association check in co-docs-builder.yml #3938

merged 1 commit into from
May 28, 2024

Conversation

scottybollinger
Copy link
Contributor

It turns out that for Elastic, github.event.pull_request.author_association == 'NONE', so this won't work

@scottybollinger
Update co-docs-builder.yml
099a7af 
It turns out that for Elastic,  github.event.pull_request.author_association == 'NONE', so this won't work
@scottybollinger scottybollinger requested a review from a team as a code owner May 28, 2024 22:12
@scottybollinger scottybollinger enabled auto-merge (squash) May 28, 2024 22:12
@mergify Mergify
Copy link
Contributor

mergify bot commented May 28, 2024

This pull request does not have a backport label. Could you fix it @scottybollinger? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-/d./d is the label to automatically backport to the /d./d branch. /d is the digit
    NOTE: backport-skip has been added to this pull request.

@mergify mergify bot added the backport-skip Skip notification from the automated backport with mergify label May 28, 2024
@scottybollinger scottybollinger changed the title Update co-docs-builder.yml Remove author_association check in co-docs-builder.yml May 28, 2024
@scottybollinger scottybollinger merged commit ceccfb2 into main May 28, 2024
9 checks passed
@scottybollinger scottybollinger deleted the scottybollinger-patch-2 branch May 28, 2024 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@glitteringkatie glitteringkatie glitteringkatie approved these changes

@colleenmcginnis colleenmcginnis Awaiting requested review from colleenmcginnis colleenmcginnis is a code owner automatically assigned from elastic/docs-engineering

Assignees
No one assigned
Labels
backport-skip Skip notification from the automated backport with mergify
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@scottybollinger @glitteringkatie