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

bitrise-step-appdome-build-2secure-ios 1.0.2 #3808

Merged
merged 1 commit into from
Apr 28, 2023
Merged

bitrise-step-appdome-build-2secure-ios 1.0.2 #3808

merged 1 commit into from
Apr 28, 2023

Conversation

elish266
Copy link
Contributor

@elish266 elish266 commented Apr 23, 2023

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.

@elish266 elish266 mentioned this pull request Apr 23, 2023
5 tasks
@bitrise-coresteps-bot
Copy link
Collaborator

Don't forget to follow semantic versioning! Breaking changes (including step inputs) should be in major versions only.

step.yml changes compared to previous version:

summary: |
Builds an iOS mobile app using Appdome's platform
description: |
Integration that allows activating security and app protection features, bulding and signing mobile apps using Appdome's API. For detailes see: https://www.appdome.com/how-to/appsec-release-orchestration/mobile-appsec-cicd/use-appdome-build-2secure-step-for-bitrise
Copy link
Contributor

Choose a reason for hiding this comment

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

Note: the URL is not available (yet)

Suggested change
Integration that allows activating security and app protection features, bulding and signing mobile apps using Appdome's API. For detailes see: https://www.appdome.com/how-to/appsec-release-orchestration/mobile-appsec-cicd/use-appdome-build-2secure-step-for-bitrise
Integration that allows activating security and app protection features, bulding and signing mobile apps using Appdome's API. For details see: https://www.appdome.com/how-to/appsec-release-orchestration/mobile-appsec-cicd/use-appdome-build-2secure-step-for-bitrise

Copy link
Contributor

@lpusok lpusok left a comment

Choose a reason for hiding this comment

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

Would recommend adding an input for APPDOME_API_KEY in the next release.
See an example here: https://github.com/bitrise-steplib/steps-google-play-deploy/blob/2049e4237946b3ffd8f557346b7e8fc1438d3f07/step.yml#LL71C1-L71C1.

Left a note about the description.

@lpusok lpusok merged commit 0335ad1 into bitrise-io:master Apr 28, 2023
@lpusok
Copy link
Contributor

lpusok commented Apr 28, 2023

merged.

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.

3 participants