Skip to content

[DO NOT MERGE] Test prow presubmit#77

Closed
adrcunha wants to merge 2 commits into
masterfrom
adrcunha-patch-1
Closed

[DO NOT MERGE] Test prow presubmit#77
adrcunha wants to merge 2 commits into
masterfrom
adrcunha-patch-1

Conversation

@adrcunha
Copy link
Copy Markdown
Contributor

@adrcunha adrcunha commented Jun 7, 2018

No description provided.

@google-prow-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adrcunha

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-prow-robot google-prow-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jun 7, 2018
@adrcunha
Copy link
Copy Markdown
Contributor Author

adrcunha commented Jun 8, 2018

/test pull-knative-eventing-integration-tests

@adrcunha
Copy link
Copy Markdown
Contributor Author

/hold

@google-prow-robot google-prow-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jun 11, 2018
@adrcunha
Copy link
Copy Markdown
Contributor Author

/test all

@adrcunha
Copy link
Copy Markdown
Contributor Author

Required checks are now enforced.

@adrcunha adrcunha closed this Jun 12, 2018
@adrcunha adrcunha mentioned this pull request Jun 12, 2018
@mattmoor mattmoor deleted the adrcunha-patch-1 branch August 3, 2018 03:53
matzew pushed a commit to matzew/eventing that referenced this pull request Apr 30, 2019
matzew pushed a commit to matzew/eventing that referenced this pull request Feb 10, 2023
Whenever I set additional PingSource adapter env variables, the
controller reverts my changes since it's doing a `DeepEqual` on
env variables instead of doing `DeepDerivative(expected, current).

Signed-off-by: Pierangelo Di Pilato <pierdipi@redhat.com>

Signed-off-by: Pierangelo Di Pilato <pierdipi@redhat.com>
Co-authored-by: Pierangelo Di Pilato <pierdipi@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants