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

Github Actions dawidd6/action-download-artifact set workflow_conclusion #1441

Merged
merged 1 commit into from
Mar 16, 2021

Conversation

Scottmitch
Copy link
Member

Motivation:
ec26d27 updated
dawidd6/action-download-artifact which changed the default behavior of
workflow_conclusion from "completed" to "completed,success". This may
result in failure of downloading test results for pull request builder.

Modifications:

  • explicitly set workflow_conclusion to "completed"

Result:
Unit test and quality results are downloaded to generate reports.

Motivation:
ec26d27 updated
dawidd6/action-download-artifact which changed the default behavior of
workflow_conclusion from "completed" to "completed,success". This may
result in failure of downloading test results for pull request builder.

Modifications:
- explicitly set workflow_conclusion to "completed"

Result:
Unit test and quality results  are downloaded to generate reports.
@Scottmitch
Copy link
Member Author

tested on my local fork (https://github.com/Scottmitch/servicetalk-1/runs/2125698762?check_suite_focus=true), however may require followup...

@Scottmitch
Copy link
Member Author

will merge so I can see test errors in other PRs, will followup as necessary.

@Scottmitch Scottmitch merged commit 627c354 into apple:main Mar 16, 2021
@Scottmitch Scottmitch deleted the gha_prb_reports_download branch March 16, 2021 22:03
bondolo pushed a commit to bondolo/servicetalk that referenced this pull request Mar 16, 2021
…on (apple#1441)

Motivation:
ec26d27 updated
dawidd6/action-download-artifact which changed the default behavior of
workflow_conclusion from "completed" to "completed,success". This may
result in failure of downloading test results for pull request builder.

Modifications:
- explicitly set workflow_conclusion to "completed"

Result:
Unit test and quality results  are downloaded to generate reports.
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.

1 participant