feat: test pr #855
feat: test pr #855
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
![ellipsis-dev[bot]](https://avatars.githubusercontent.com/in/64358?s=60&v=4){kind=small}
There was a problem hiding this comment.
❌ Changes requested. Reviewed everything up to c500df7 in 48 seconds
More details
- Looked at
14lines of code in1files - Skipped
0files when reviewing. - Skipped posting
0drafted comments based on config settings.
Workflow ID: wflow_dMlwhHipeaSipzUB
Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.
| raise ComposioClientError("Error fetching client ID") | ||
|
|
||
| print( | ||
| f"{client_id=} base_url={self.client.http.base_url} api_key={self.client.api_key}" |
There was a problem hiding this comment.
Remove the print statement or replace it with proper logging to avoid exposing sensitive information.
| f"{client_id=} base_url={self.client.http.base_url} api_key={self.client.api_key}" | |
| self.logger.info(f"{client_id=} base_url={self.client.http.base_url} api_key={self.client.api_key}") |
|
This comment was generated by github-actions[bot]! JS SDK Coverage Report📊 Coverage report for JS SDK can be found at the following URL: 📁 Test report folder can be found at the following URL: |
Important
Add print statement in
subscribe()incollections.pyfor debuggingclient_id,base_url, andapi_key.subscribe()incollections.pyto outputclient_id,base_url, andapi_key.This description was created by
for c500df7. It will automatically update as commits are pushed.