Skip to content

Add Ankitsinghsisodya as a member to knative and knative-extensions#1786

Merged
knative-prow[bot] merged 2 commits into
knative:mainfrom
Ankitsinghsisodya:add-Ankitsinghsisodya-as-member
Jun 11, 2026
Merged

Add Ankitsinghsisodya as a member to knative and knative-extensions#1786
knative-prow[bot] merged 2 commits into
knative:mainfrom
Ankitsinghsisodya:add-Ankitsinghsisodya-as-member

Conversation

@Ankitsinghsisodya

Copy link
Copy Markdown
Contributor

Changes

Adding myself as a Knative and knative-extensions member so I don't need to wait for the ok-to-test labels.

My Contributions

knative/func (PRs) – Core Contributor

All PRs: https://github.com/knative/func/pulls?q=is%3Apr+author%3AAnkitsinghsisodya+is%3Amerged

knative/eventing (PRs)

All PRs: https://github.com/knative/eventing/pulls?q=is%3Apr+author%3AAnkitsinghsisodya+is%3Amerged

knative/serving (PRs)

All PRs: https://github.com/knative/serving/pulls?q=is%3Apr+author%3AAnkitsinghsisodya+is%3Amerged

Copilot AI review requested due to automatic review settings June 9, 2026 12:55
@knative-prow knative-prow Bot requested review from evankanderson and matzew June 9, 2026 12:55
@knative-prow

knative-prow Bot commented Jun 9, 2026

Copy link
Copy Markdown

Welcome @Ankitsinghsisodya! It looks like this is your first PR to knative/community 🎉

@knative-prow knative-prow Bot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Jun 9, 2026
@knative-prow

knative-prow Bot commented Jun 9, 2026

Copy link
Copy Markdown

Hi @Ankitsinghsisodya. Thanks for your PR.

I'm waiting for a knative member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Tip

We noticed you've done this a few times! Consider joining the org to skip this step and gain /lgtm and other bot rights. We recommend asking approvers on your previous PRs to sponsor you.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

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-sigs/prow repository.

@knative-prow knative-prow Bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jun 9, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds a new GitHub organization member to the peribolos configuration for both Knative orgs.

Changes:

  • Add Ankitsinghsisodya to peribolos/knative.yaml members list
  • Add Ankitsinghsisodya to peribolos/knative-extensions.yaml members list

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
peribolos/knative.yaml Adds a new org member entry to the members list
peribolos/knative-extensions.yaml Mirrors the same new org member entry for knative-extensions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread peribolos/knative.yaml Outdated
Comment thread peribolos/knative-extensions.yaml Outdated
@Ankitsinghsisodya

Ankitsinghsisodya commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

@evankanderson evankanderson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm
approve

@knative-prow knative-prow Bot added the lgtm Indicates that a PR is ready to be merged. label Jun 10, 2026
@Ankitsinghsisodya

Copy link
Copy Markdown
Contributor Author

@lkingland

lkingland commented Jun 10, 2026

Copy link
Copy Markdown
Member

/approve
/lgtm

@Ankitsinghsisodya

Copy link
Copy Markdown
Contributor Author

/lgtm approve

Hey @evankanderson, could you run /ok-to-test and /approve (with the slash prefix) so the bot can pick them up? It looks like your previous approve comment missed the /.

@evankanderson

Copy link
Copy Markdown
Member

/ok-to-test
/approve

I tried this from GitHub Mobile, but it seems like that (or GitHub's backend) was having a bad day, and it ate the comment.

@knative-prow knative-prow Bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Jun 11, 2026
@knative-prow

knative-prow Bot commented Jun 11, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Ankitsinghsisodya, evankanderson, lkingland

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

@knative-prow knative-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 11, 2026
@knative-prow knative-prow Bot merged commit 923e1d6 into knative:main Jun 11, 2026
6 checks passed
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. lgtm Indicates that a PR is ready to be merged. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. 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.

4 participants