Use target.service in error messages. · RTDataScience/docker-python@bef0ead · GitHub

RTDataScience / docker-python Public

forked from Kaggle/docker-python

• Notifications You must be signed in to change notification settings
• Fork 0
• Star 0

• Code
• Pull requests
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit bef0ead

Browse filesBrowse the repository at this point in the historyBrowse files

ifigotin

committed

Use target.service in error messages.

1 parent 125c75e commit bef0ead
Copy full SHA for bef0ead

1 file changed

+3-4Lines changed: 3 additions & 4 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• patches
  • kaggle_gcp.py

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎patches/kaggle_gcp.py‎

Copy file name to clipboardExpand all lines: patches/kaggle_gcp.py

+3-4Lines changed: 3 additions & 4 deletions

Original file line number	Diff line number	Diff line change
@@ -57,10 +57,9 @@ def refresh(self, request):
57	57	raise RefreshError('Unable to refresh access token due to connection error.') from e
58	58	except Exception as e:
59	59	Log.error(f"Error trying to refresh access token: {e}")
60		- if (not get_integrations().has_bigquery() and self.target == GcpTarget.BIGQUERY):
61		- Log.error(f"No bigquery integration found.")
62		- print(
63		- 'Please ensure you have selected a BigQuery account in the Kernels Settings sidebar.')
	60	+ Log.error(f"No {self.target.service} integration found.")
	61	+ print(
	62	+ f"Please ensure you have selected a {self.target.service} account in the Kernels Settings sidebar.")
64	63	raise RefreshError('Unable to refresh access token.') from e
65	64
66	65

0 commit comments

Comments

0 (0)