Skip to content

Move sbt override to cloud build#637

Open
mkolodner-sc wants to merge 2 commits into
mainfrom
mkolodner-sc/fix_sbt_cloud_build
Open

Move sbt override to cloud build#637
mkolodner-sc wants to merge 2 commits into
mainfrom
mkolodner-sc/fix_sbt_cloud_build

Conversation

@mkolodner-sc
Copy link
Copy Markdown
Collaborator

Scope of work done

Where is the documentation for this feature?: N/A

Did you add automated tests or write a test plan?

Updated Changelog.md? NO

Ready for code review?: NO

@mkolodner-sc mkolodner-sc changed the title [WIP] full sbt override to cloud [WIP] full sbt override to cloud build May 13, 2026
@mkolodner-sc mkolodner-sc changed the title [WIP] full sbt override to cloud build [WIP] move sbt override to cloud build May 13, 2026
@mkolodner-sc
Copy link
Copy Markdown
Collaborator Author

/unit_test_scala

@mkolodner-sc
Copy link
Copy Markdown
Collaborator Author

/lint_test

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 13, 2026

GiGL Automation

@ 01:38:12UTC : 🔄 Scala Unit Test started.

@ 01:46:58UTC : ✅ Workflow completed successfully.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 13, 2026

GiGL Automation

@ 01:38:16UTC : 🔄 Lint Test started.

@ 01:43:23UTC : ❌ Workflow failed.
Please check the logs for more details.

@mkolodner-sc mkolodner-sc changed the title [WIP] move sbt override to cloud build Move sbt override to cloud build May 13, 2026
@mkolodner-sc
Copy link
Copy Markdown
Collaborator Author

lint_test doesn't set use_cloud_run to True (default False), meaning the changes in this PR can't be triggered on comment for this test, and will only be seen on merge. We see through the success of the scala unit test that the change is captured.

@mkolodner-sc mkolodner-sc marked this pull request as ready for review May 13, 2026 19:20
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.

1 participant