Skip to content

Fix import for integration test#522

Merged
vish-cs merged 1 commit into
datacommonsorg:masterfrom
vish-cs:itest
Jun 5, 2026
Merged

Fix import for integration test#522
vish-cs merged 1 commit into
datacommonsorg:masterfrom
vish-cs:itest

Conversation

@vish-cs
Copy link
Copy Markdown
Contributor

@vish-cs vish-cs commented Jun 5, 2026

No description provided.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request simplifies the import setup in spanner_ingestion_test.py by appending the current directory to sys.path and importing cloud_workflow directly. Feedback suggests using os.path.abspath to ensure a robust absolute path is added to sys.path, which prevents potential import failures if the execution directory changes.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread pipeline/workflow/spanner_ingestion_test.py Outdated
@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented Jun 5, 2026

Not up to standards ⛔

🔴 Issues 7 minor

Alerts:
⚠ 7 issues (≤ 0 issues of at least minor severity)

Results:
7 new issues

Category Results
Documentation 7 minor

View in Codacy

🟢 Metrics 5 complexity · 0 duplication

Metric Results
Complexity 5
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@vish-cs vish-cs requested review from ajaits and gmechali June 5, 2026 13:03
@vish-cs vish-cs enabled auto-merge (squash) June 5, 2026 13:08
@vish-cs vish-cs merged commit 1dfd510 into datacommonsorg:master Jun 5, 2026
9 of 10 checks passed
gmechali pushed a commit to gmechali/import that referenced this pull request Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants