Closed JustinGrilli closed 1 year ago
@JustinGrilli I'm trying to troubleshoot the weird PR thing. Can you merge this in? That feels like it might make the comparison simpler but perhaps I'm wrong 😮💨
@JustinGrilli could you re-run the CI job on this branch? I'm confused as to why this would suddenly fail. If you re-run the CI job here that will let us know if the issue is the other branches OR if it's something in the CI flow from imports/versions etc.
When I google the errors on the other branchs I'm getting package version and pytest version issues. This is an old comment but perhaps it;s relevant https://github.com/pytest-dev/pytest-runner/issues/49#issuecomment-495259140
Yeah I will merge this in.
Once I merge it, we should see CI run on the refactor-to-use-restapi PR
Changes
connection
CLI command into aconnection
argument groupconnection
CLI command logic todatasource
andserver_operate
enforce_connection
with theconnection
args to update the connection in the local datasourceconn_user
andconn_pw
when calling--publish datasource
to embed creds for the connection when publishingembed_connection
withconnection
args to embed connection creds for a datasource onlineembed_connection
callserver_operate --publish
, to open the datasource's URL in the browser once published (Like it would when publishing from Tableau Desktop)server_operate
test_tableau_utilities.py
--version
argument to the CLI, which will reference the version of the installed packagereadme
for connection changesample_settings.yaml
and moved to top level folder.cli.py
help comments.--debugging_logs
is provided, it will warn the user.Notes to reviewer
connection
betweendatasource
andserver_operate
is because:Tests
After installing
tableau_utilities
with these changes:tableau_utilities --version
tableau_utilities -d -l online -n Jobs -pn "Official Datasources - Business" -tds datasource --enforce_connection
tableau_utilities -d -n Jobs -pn "Official Datasources - Business" --file_path Jobs.tdsx server_operate --publish datasource