KEMBAR78
Delete workflow confirmation check in CLI - #114 by atalhens · Pull Request #137 · NVIDIA/NeMo-Agent-Toolkit · GitHub
Skip to content

Conversation

@atalhens
Copy link
Contributor

Description

Add a confirmation check from CLI before deletion of workflow.

Closes - #114

By Submitting this PR I confirm:

  • I am familiar with the Contributing Guidelines.
  • We require that all contributors "sign-off" on their commits. This certifies that the contribution is your original work, or you have rights to submit it under the same license, or a compatible license.
    • Any contribution which contains commits that are not Signed-Off will not be accepted.
  • When the PR is ready for review, new or existing tests cover these changes.
  • When the PR is ready for review, the documentation is up to date with these changes.

Signed-off-by: atalhens <sneh.lata@nutanix.com>
@atalhens atalhens requested a review from a team as a code owner April 19, 2025 20:44
@copy-pr-bot
Copy link

copy-pr-bot bot commented Apr 19, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@atalhens
Copy link
Contributor Author

cc: @dagardner-nv, I can't find access to add labels here, which is causing Label Checker to fail. Can you guide me through if I am doing anything wrong?

@atalhens atalhens changed the title Delete workflow confirmation check in cli - #114 Delete workflow confirmation check in CLI - #114 Apr 21, 2025
@mdemoret-nv mdemoret-nv added improvement Improvement to existing functionality non-breaking Non-breaking change labels Apr 23, 2025
@mdemoret-nv
Copy link
Collaborator

/ok to test fe24c46

@mdemoret-nv
Copy link
Collaborator

Thanks for the changes!

@mdemoret-nv
Copy link
Collaborator

/merge

@rapids-bot rapids-bot bot merged commit b5ca8ea into NVIDIA:develop Apr 23, 2025
10 checks passed
ericevans-nv pushed a commit to ericevans-nv/agent-iq that referenced this pull request Apr 23, 2025
Add a confirmation check from CLI before deletion of workflow.

Closes - NVIDIA#114

## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/NVIDIA/AgentIQ/blob/develop/docs/source/advanced/contributing.md).
- We require that all contributors "sign-off" on their commits. This certifies that the contribution is your original work, or you have rights to submit it under the same license, or a compatible license.
  - Any contribution which contains commits that are not Signed-Off will not be accepted.
- When the PR is ready for review, new or existing tests cover these changes.
- When the PR is ready for review, the documentation is up to date with these changes.

Authors:
  - Snehlata (https://github.com/atalhens)

Approvers:
  - Michael Demoret (https://github.com/mdemoret-nv)

URL: NVIDIA#137
Signed-off-by: Eric Evans <194135482+ericevans-nv@users.noreply.github.com>
ericevans-nv pushed a commit to ericevans-nv/agent-iq that referenced this pull request Apr 23, 2025
Add a confirmation check from CLI before deletion of workflow.

Closes - NVIDIA#114

## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/NVIDIA/AgentIQ/blob/develop/docs/source/advanced/contributing.md).
- We require that all contributors "sign-off" on their commits. This certifies that the contribution is your original work, or you have rights to submit it under the same license, or a compatible license.
  - Any contribution which contains commits that are not Signed-Off will not be accepted.
- When the PR is ready for review, new or existing tests cover these changes.
- When the PR is ready for review, the documentation is up to date with these changes.

Authors:
  - Snehlata (https://github.com/atalhens)

Approvers:
  - Michael Demoret (https://github.com/mdemoret-nv)

URL: NVIDIA#137
Signed-off-by: Eric Evans <194135482+ericevans-nv@users.noreply.github.com>
yczhang-nv pushed a commit to yczhang-nv/NeMo-Agent-Toolkit that referenced this pull request May 8, 2025
Add a confirmation check from CLI before deletion of workflow.

Closes - NVIDIA#114

## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/NVIDIA/AgentIQ/blob/develop/docs/source/advanced/contributing.md).
- We require that all contributors "sign-off" on their commits. This certifies that the contribution is your original work, or you have rights to submit it under the same license, or a compatible license.
  - Any contribution which contains commits that are not Signed-Off will not be accepted.
- When the PR is ready for review, new or existing tests cover these changes.
- When the PR is ready for review, the documentation is up to date with these changes.

Authors:
  - Snehlata (https://github.com/atalhens)

Approvers:
  - Michael Demoret (https://github.com/mdemoret-nv)

URL: NVIDIA#137
Signed-off-by: Yuchen Zhang <134643420+yczhang-nv@users.noreply.github.com>
AnuradhaKaruppiah pushed a commit to AnuradhaKaruppiah/oss-agentiq that referenced this pull request Aug 4, 2025
Add a confirmation check from CLI before deletion of workflow.

Closes - NVIDIA#114 

## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/NVIDIA/AgentIQ/blob/develop/docs/source/advanced/contributing.md).
- We require that all contributors "sign-off" on their commits. This certifies that the contribution is your original work, or you have rights to submit it under the same license, or a compatible license.
  - Any contribution which contains commits that are not Signed-Off will not be accepted.
- When the PR is ready for review, new or existing tests cover these changes.
- When the PR is ready for review, the documentation is up to date with these changes.

Authors:
  - Snehlata (https://github.com/atalhens)

Approvers:
  - Michael Demoret (https://github.com/mdemoret-nv)

URL: NVIDIA#137
scheckerNV pushed a commit to scheckerNV/aiq-factory-reset that referenced this pull request Aug 22, 2025
Add a confirmation check from CLI before deletion of workflow.

Closes - NVIDIA#114 

## By Submitting this PR I confirm:
- I am familiar with the [Contributing Guidelines](https://github.com/NVIDIA/AgentIQ/blob/develop/docs/source/advanced/contributing.md).
- We require that all contributors "sign-off" on their commits. This certifies that the contribution is your original work, or you have rights to submit it under the same license, or a compatible license.
  - Any contribution which contains commits that are not Signed-Off will not be accepted.
- When the PR is ready for review, new or existing tests cover these changes.
- When the PR is ready for review, the documentation is up to date with these changes.

Authors:
  - Snehlata (https://github.com/atalhens)

Approvers:
  - Michael Demoret (https://github.com/mdemoret-nv)

URL: NVIDIA#137
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement to existing functionality non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants