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

Apps decompiler 1.0.0 #2865

Closed
wants to merge 3 commits into from

Conversation

imranMnts
Copy link
Contributor

@imranMnts imranMnts commented Jan 21, 2021

TagCheck

What to do if the build fails?

At the moment contributors do not have access to the CI workflow triggered by StepLib PRs. In case of a failed build, we ask for your patience. Maintainers of Bitrise Steplib will sort it out for you or inform you if any further action is needed.

New Pull Request Checklist

Please mark the points which you did / accept.

New Step
Thank you for the new Step share! The CI check might will fail due to our extended validation engine. Nothing to worry about yet, we will get back to you shortly.

- ios_ipa_path: $BITRISE_IPA_PATH
opts:
description: |
If you use this step in the same build where you generated the IPA
Copy link
Contributor

@koral-- koral-- Jan 21, 2021

Choose a reason for hiding this comment

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

The wording sounds strange. Is it a calque?
It's just an IPA path? What is the point of "If"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The user will be able to launch this step from the step where he will generate his IPA
In that case, we need to have the path to get it. Per default XCode archive step save the path in $BITRISE_IPA_PATH but to be able to support the case where the user deployed it in another path we have this input.

In the case where the user launches this step outside of the step where he will generate the IPA (eg. to launch a build specifically for quality to not mix it with deploy build) also we don't need to have this path because it will download the IPA directly from the deployment builds artifacts with outside_build_slug input

Copy link
Contributor

Choose a reason for hiding this comment

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

OK, so you should somehow express it in a condensed way for a valid description.

project_type_tags:
- ios
- android
- react-native
Copy link
Contributor

Choose a reason for hiding this comment

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

What about other multi-platform ones?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Based on
https://github.com/bitrise-steplib/steps-xcode-archive/blob/master/step.yml
and
https://github.com/bitrise-steplib/steps-gradle-runner/blob/master/step.yml

which generate the APK and IPA
I don't test it with the others so I don't know if it can be used, so to be sure, I didn't add them.

- name: jq
- name: apktool
is_requires_admin_user: false
is_always_run: true
Copy link

Choose a reason for hiding this comment

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

Do you want the is_always_run field to be true?

is_always_run: if true the step will be executed even if a previous step failed during the build. 
Default is false.
- if type_tags is 'notification' this needs to be true

Copy link

Choose a reason for hiding this comment

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

I think it is not necessary for this step :)

@imranMnts imranMnts closed this Jan 25, 2021
@imranMnts
Copy link
Contributor Author

Continues in #2874

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.

None yet

3 participants