Skip to content
This repository was archived by the owner on Sep 10, 2025. It is now read-only.

Conversation

@marcin-kozinski
Copy link
Collaborator

I noticed I was calling bash from a kotlin script called from bash. Instead refactored the kotlin script to be more focussed on the task and added the conditionals that used bash to the calling bash script.

References

PR Checklist

Setup:

  • Described changes for automated release notes in PR title using
    Conventional Commits standard
  • Self Review (review, clean up, documentation)
  • Basic Self QA

Review:

  • Code Review approved

@marcin-kozinski marcin-kozinski added the build-setup Move to "Build setup updates" section in release notes. label Jan 8, 2025
@marcin-kozinski marcin-kozinski added this pull request to the merge queue Jan 8, 2025
Merged via the queue into main with commit 56a08cc Jan 8, 2025
5 checks passed
@marcin-kozinski marcin-kozinski deleted the avoid-bash-kotlin-bash-sandwich branch January 8, 2025 12:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

build-setup Move to "Build setup updates" section in release notes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants