Skip to content

Commit

Permalink
test bitrise.yml fix
Browse files Browse the repository at this point in the history
  • Loading branch information
viktorbenei committed Nov 3, 2015
1 parent 67a81c4 commit f3b7cc5
Showing 1 changed file with 15 additions and 18 deletions.
33 changes: 15 additions & 18 deletions bitrise.yml
Original file line number Diff line number Diff line change
@@ -1,33 +1,30 @@
format_version: 0.9.10
format_version: 1.1.0
default_step_lib_source: https://github.com/bitrise-io/bitrise-steplib.git

app:
envs:
- api_key: null
- BITRISE_IPA_PATH: null
- BITRISE_DSYM_PATH: null
# define these in your .bitrise.secrets.yml
- api_key:
- BITRISE_IPA_PATH:
- BITRISE_DSYM_PATH:

workflows:
test:
steps:
- path::./:
inputs:
- notify: "off"
opts: {}
- auto_update: "off"
opts: {}
- max_test_duration: 10m
opts: {}
- opts: {}
video_recording: "off"
- max_test_duration: "10m"
- video_recording: "off"
- comment: ""
opts: {}
- opts: {}
tester_groups: ""
outputs:
- TESTFAIRY_PUBLIC_INSTALL_PAGE_URL: ""
opts:
title: Public Install Page URL
- tester_groups: ""
- script:
inputs:
- content: |-
#!/bin/bash
echo "TESTFAIRY_PUBLIC_INSTALL_PAGE_URL: ${TESTFAIRY_PUBLIC_INSTALL_PAGE_URL}"
if [[ "$TESTFAIRY_PUBLIC_INSTALL_PAGE_URL" == "" ]] ; then
echo "No TESTFAIRY_PUBLIC_INSTALL_PAGE_URL env available!!"
exit 1
fi
echo "TESTFAIRY_PUBLIC_INSTALL_PAGE_URL: ${TESTFAIRY_PUBLIC_INSTALL_PAGE_URL}"

4 comments on commit f3b7cc5

@guitcastro
Copy link

Choose a reason for hiding this comment

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

This part should be kept, no ?

        - TESTFAIRY_PUBLIC_INSTALL_PAGE_URL: ""     
          opts:     
            title: Public Install Page URL

@viktorbenei
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not required to have it in the bitrise.yml @guitcastro , it have to be in the step.yml but you only have to list the things you want to change in the bitrise.yml.

@guitcastro
Copy link

Choose a reason for hiding this comment

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

@viktorbenei I just realised this is not the step.yml file. Sorry I should pay more attention ;p

@viktorbenei
Copy link
Contributor Author

Choose a reason for hiding this comment

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

No problem at all 😀

Please sign in to comment.