Move sbt override to cloud build

[mkolodner-sc]

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]

/unit_test_scala

[mkolodner-sc]

/lint_test

[github-actions]

GiGL Automation

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

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

[github-actions]

GiGL Automation

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

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

[mkolodner-sc]

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.

[kmontemayor2-sc]

Thanks matt! Can we ad comments to these overrides saying why we do them?

[svij-sc]

I would personally do:

cp path/to/repositories ~/.sbt/repositories once in run_command_on_active_checkout, then remove the -Dsbt.repository.config=repositories from .sbtopts file.
Or, just bake the repositories file inside the docker image for cloud build.

Then we have one source of truth not two.

But, this is fine too given we will deprecate scala code soon anyways.