Conversation
| @pytest.mark.dependency(depends=["test_list_projects"]) | ||
| def test_list_users(): | ||
| _test_cloud_cli( | ||
| [ | ||
| "users", | ||
| "list", | ||
| "--account-id", | ||
| ACCOUNT_ID, | ||
| ], | ||
| "user_id", | ||
| ) | ||
|
|
||
|
|
There was a problem hiding this comment.
We needed this additional test for using pytest.user_id in the notifications tests
|
Hey @dpguthrie, let me know if I can do anything here! |
dbtc/_version.py
Outdated
| @@ -1 +1 @@ | |||
| __version__ = "0.11.6" | |||
| __version__ = "0.12.0" | |||
There was a problem hiding this comment.
This can prolly just be 0.11.7 as there are no breaking changes (unless I missed something). Other than that, looks good (and so sorry that it's taken so long for me to review).
|
@nchylak apologize for being so late here - just had one comment. Should be good other than that. |
No problem! The version is adjusted :) |
|
Thanks! Could you please also release this new version so that we can use it at my company 🙏 ? |
0.11.7 should be installable via pypi now. |
2 participants
Implements dbt Cloud's API notifications endpoint as discussed in issue #111
The changes were tested as detailed in the pytest files, using our company dbt Cloud's account and were successful on our side :)