Skip to content

Conversation

vdemeester
Copy link
Member

@vdemeester vdemeester commented May 6, 2025

Vibhav has been consistently contributing to pipelines for a long
while now. From the debug features to the recent "maintenance" task of
removing deprecated features (ClusterTask) and promoting others (StepActions).

Along the way, he has racked up well over a 200+ reviews:

https://github.com/tektoncd/pipeline/pulls?q=is%3Apr++reviewed-by%3Awaveywaves
https://github.com/tektoncd/pipeline/pulls?q=is%3Apr++author%3Awaveywaves+

Signed-off-by: Vincent Demeester [email protected]

Vibhav has been consistently contributing to pipelines for a long
while now. From the debug features to the recent "maintenance" task of
removing deprecated features (ClusterTask) and promoting others (StepActions).

Along the way, he has racked up well over a 200+ reviews:

https://github.com/tektoncd/pipeline/pulls?q=is%3Apr++reviewed-by%3AJeromeJu+-author%3Awaveywaves+

Signed-off-by: Vincent Demeester <[email protected]>
@tekton-robot tekton-robot added do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels May 6, 2025
@vdemeester
Copy link
Member Author

/release-note-none
/kind misc

@tekton-robot tekton-robot added release-note-none Denotes a PR that doesnt merit a release note. kind/misc Categorizes issue or PR as a miscellaneuous one. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels May 6, 2025
@vdemeester
Copy link
Member Author

@abayer
Copy link
Contributor

abayer commented May 6, 2025

/approve

@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 6, 2025
@afrittoli
Copy link
Member

Thanks for this @vdemeester - I'd love to see @waveywaves joining the maintainers team.
Thank @waveywaves for all your contributions so far!
/approve

@chitrangpatel
Copy link
Contributor

/approve

@tekton-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abayer, afrittoli, chitrangpatel

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:
  • OWNERS [abayer,afrittoli,chitrangpatel]

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

@abayer
Copy link
Contributor

abayer commented May 6, 2025

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label May 6, 2025
@tekton-robot tekton-robot merged commit d955d01 into tektoncd:main May 6, 2025
20 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in Tekton Community Roadmap May 6, 2025
@vdemeester vdemeester deleted the add-waveywaves-as-maintainer branch May 6, 2025 15:49
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. kind/misc Categorizes issue or PR as a miscellaneuous one. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesnt merit a release note. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants