Skip to content
This repository has been archived by the owner on Dec 5, 2024. It is now read-only.

Add Triggers 0.5.0 support for webhooks extension #549

Merged
merged 1 commit into from
May 26, 2020

Conversation

a-roberts
Copy link
Member

@a-roberts a-roberts commented May 26, 2020

Closes #547

Important note: hold on, let's see what we hear regarding tektoncd/triggers#583 as obviously the steps here aren't ideal

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

See the contribution guide
for more details.

@tekton-robot tekton-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label May 26, 2020
@a-roberts a-roberts added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 26, 2020
@a-roberts
Copy link
Member Author

We've had an update back regarding the Triggers issue I made, it was an undocumented/unsupported field we were using anyway so this PR should be good to go now.

@a-roberts a-roberts removed do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels May 26, 2020
@a-roberts a-roberts requested review from AlanGreene and eddycharly and removed request for akihikokuroda May 26, 2020 15:08
Copy link
Member

@AlanGreene AlanGreene left a comment

Choose a reason for hiding this comment

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

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label May 26, 2020
@tekton-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: AlanGreene

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

The pull request process is described here

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

@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 26, 2020
@tekton-robot tekton-robot merged commit 85f9f36 into tektoncd:master May 26, 2020
@a-roberts a-roberts deleted the triggers050upgrade branch May 26, 2020 15:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Triggers 0.5 support for webhooks extension
3 participants