Skip to content
This repository was archived by the owner on Apr 23, 2025. It is now read-only.

feat: Change job trigger to pull_request_target #596

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

richardkocian
Copy link
Contributor

What is the purpose of this change? What does it change?

Triggering job Java CI with Gradle using pull_request_target instead of pull_request.

Was the change discussed in an issue?

This change is needed for correct function of GitHub secrets, which are used in #563

@richardkocian richardkocian added the enhancement New feature or request label Oct 16, 2023
@richardkocian richardkocian self-assigned this Oct 16, 2023
@openshift-ci openshift-ci bot requested review from adietish and sbouchet October 16, 2023 11:43
@richardkocian richardkocian removed the request for review from adietish October 16, 2023 11:43
@richardkocian richardkocian changed the title Change job trigger to pull_request_target feat: Change job trigger to pull_request_target Oct 16, 2023
@openshift-ci
Copy link

openshift-ci bot commented Oct 16, 2023

@richard0202: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-4.11 dbb5643 link true /test e2e-4.11

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Copy link
Collaborator

@sbouchet sbouchet left a comment

Choose a reason for hiding this comment

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

LGTM, let's see how it goes .

@openshift-ci openshift-ci bot added the lgtm label Oct 16, 2023
@sbouchet sbouchet merged commit 20b10dc into redhat-developer:main Oct 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request lgtm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants