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

Add outputs for schedule run #17

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

mina-kleid
Copy link

Summary

In response to this Issue about test-application is failing without any outputs

Introduced changes

  • Add run arn as an output once it becomes available after scheduleRun
  • Add result as an output in case the run has not passed

@cla-bot
Copy link

cla-bot bot commented Oct 2, 2023

Realm welcomes all contributions! The only requirement we have is that, like many other projects, we need to have a Contributor License Agreement (CLA) in place before we can accept any external code. Our own CLA is a modified version of the Apache Software Foundation’s CLA. Our records show that CLA has not been signed by @mina-kleid. Please submit your CLA electronically using our Google form so we can accept your submissions. After signing the CLA you can recheck this PR with a @cla-bot check comment. The GitHub usernames you file there will need to match that of your Pull Requests. If you have any questions or cannot file the CLA electronically, make a comment here and we will be happy to help you out.

@mina-kleid
Copy link
Author

@cla-bot check

@cla-bot cla-bot bot added the cla: yes label Oct 2, 2023
@cla-bot
Copy link

cla-bot bot commented Oct 2, 2023

The cla-bot has been summoned, and re-checked this pull request!

@bwachter
Copy link
Collaborator

bwachter commented Oct 2, 2023

You'll need to run npm run package, which should then also change test-application/dist/index.js

@mina-kleid
Copy link
Author

@bwachter Added the npm run package b27baf1

@bwachter
Copy link
Collaborator

bwachter commented Oct 2, 2023

I'll need to change the integration workflow to allow running with devicefarm credential access after manual review for this to pass. I'll probably get to that tomorrow.

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.

2 participants