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 support of specifying branch of the pipeline + ability of waiting for the results #81

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

Conversation

LA-Toth
Copy link

@LA-Toth LA-Toth commented Apr 13, 2023

Based on v1.2

The original implementation didn't wait for the results of the started pipeline, and only the default branch was supported.

The action can wait for the pipeline result, and if it's passed, the action is passed, otherwise it's failed.
The results are checked in every X seconds, which is configurable and the default is 10 seconds.
This feature blocks the currently used GitHub runner.

The branch of the pipeline can also be specified.

I also made code cleanup, because the tests didn't work at all, and the code now uses UNIX new lines.

Nishu Bansal and others added 21 commits November 6, 2019 10:23
* Bump @actions/core from 1.1.3 to 1.2.6 (Azure#19)

Bumps [@actions/core](https://github.com/actions/toolkit/tree/HEAD/packages/core) from 1.1.3 to 1.2.6.
- [Release notes](https://github.com/actions/toolkit/releases)
- [Changelog](https://github.com/actions/toolkit/blob/main/packages/core/RELEASES.md)
- [Commits](https://github.com/actions/toolkit/commits/HEAD/packages/core)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Core module upgrade

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Updating node-api Azure#42

* Adding azure-devops-node-api

Co-authored-by: Himanshu Verma <[email protected]>
Added L0 test file
* Feature/add pipeline variables (Azure#11)

* Add pipeline variables

* Better variables handling

* Update README.md

* Update README.md

Co-authored-by: ionutleca <[email protected]>
Co-authored-by: Jyotsna <[email protected]>
Default is 10 seconds.
The time unit is seconds.
@elliotzh
Copy link

Hi @LA-Toth & @Josh-01 , do we have any plan to merge this pull request?

@LA-Toth
Copy link
Author

LA-Toth commented Aug 18, 2023

@microsoft-github-policy-service agree

@LA-Toth
Copy link
Author

LA-Toth commented Aug 18, 2023

Hi @LA-Toth & @Josh-01 , do we have any plan to merge this pull request?

Honestly, I completely forgot this PR. I think it would be great if this could be merged.

@LA-Toth
Copy link
Author

LA-Toth commented Aug 19, 2023

For me, it looks like that the master branch is not the main branch of the repo, but the releases/v1 or something like that, somehow a non-master branch has more features.
This is my branch diverged from master. But now I'm unsure what to do.

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.

6 participants