Skip to content

chore: Cli integration test #272

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 104 commits into
base: main
Choose a base branch
from
Open

chore: Cli integration test #272

wants to merge 104 commits into from

Conversation

manisha1997
Copy link
Contributor

@manisha1997 manisha1997 commented May 8, 2025

Fixes

  1. Fixed auditing vulnerabilities
  2. Updated node versions
  3. did 1-1 mapping of cli-core to twilio-cli

Checklist

  • I acknowledge that all my contributions will be made under the project's license
  • I have made a material change to the repo (functionality, testing, spelling, grammar)
  • I have read the Contribution Guidelines and my PR follows them
  • I have titled the PR appropriately
  • I have updated my branch with the main branch
  • I have added tests that prove my fix is effective or that my feature works
  • I have added the necessary documentation about the functionality in the appropriate .md file
  • I have added inline documentation to the code I modified

If you have questions, please file a support ticket, or create a GitHub Issue in this repository.

@manisha1997 manisha1997 force-pushed the cli-integration-test branch from b250fee to c0849f3 Compare May 20, 2025 09:58
Copy link

Copy link

@manisha1997 manisha1997 force-pushed the cli-integration-test branch from 2ad98d9 to 9820b03 Compare May 20, 2025 10:50
Copy link

1 similar comment
Copy link

Copy link

1 similar comment
Copy link

Copy link

Please retry analysis of this Pull-Request directly on SonarQube Cloud

Copy link

@manisha1997 manisha1997 changed the title oaiFeat: Cli integration test chore: Cli integration test May 23, 2025
- name: Execute py script to validate twilio-cli tokens
id: cli_token
run: |
output=$(python3 .github/scripts/validate_cli_tokens.py)
Copy link
Contributor

Choose a reason for hiding this comment

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

Any particular reason to remove the echo statement from logs?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants