Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: multiline string cmd for prepare_apk #23

Conversation

vladyslav-androshchuk
Copy link
Contributor

Hello!

If running the bundletool fastlane plugin on Windows cmd, you get an error - The syntax of the command is incorrect.. The reason is that putting new lines in the ruby string creates new lines in the output, and this can lead to inconsistent run in some shells.

When using \ we ensure that the resulting command is a single line.

@vladyslav-androshchuk
Copy link
Contributor Author

@MartinGonzalez Hello! Can I ask you to review this, MR? I found this one, when moved our Android building pipeline to some Windows nodes.

@MartinGonzalez MartinGonzalez changed the title Fix multiline string cmd for prepare_apk fix: multiline string cmd for prepare_apk Jul 23, 2024
@MartinGonzalez MartinGonzalez self-requested a review July 23, 2024 10:33
Copy link
Owner

@MartinGonzalez MartinGonzalez left a comment

Choose a reason for hiding this comment

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

Thanks a lot for the fix and contribution! ❤️

@MartinGonzalez MartinGonzalez merged commit ceb9fe1 into MartinGonzalez:develop Jul 23, 2024
7 of 8 checks passed
@vladyslav-androshchuk vladyslav-androshchuk deleted the fix/single-line-cmd-for-prepare-apk branch July 23, 2024 10:38
github-actions bot pushed a commit that referenced this pull request Jul 23, 2024
# 1.0.0 (2024-07-23)

### Bug Fixes

* escape keystore_info params passed to bundletool ([3651593](3651593))
* fix usage URI instead of deprecated Kernel ([3198783](3198783))
* handle special characters ([4a80855](4a80855))
* Merge pull request [#17](#17) from christianEconify/fix/handle-special-characters ([dc65f58](dc65f58))
* multiline string cmd for prepare_apk ([#23](#23)) ([ceb9fe1](ceb9fe1))
* tests on pr ([dbea065](dbea065))
github-actions bot pushed a commit that referenced this pull request Jul 23, 2024
## [1.0.11](v1.0.10...v1.0.11) (2024-07-23)

### Bug Fixes

* multiline string cmd for prepare_apk ([#23](#23)) ([ceb9fe1](ceb9fe1))
@MartinGonzalez
Copy link
Owner

🎉 This issue has been resolved in version 1.0.11 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants