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

Python: Add PR number to test coverage workflow #9964

Conversation

TaoChenOSU
Copy link
Contributor

@TaoChenOSU TaoChenOSU commented Dec 12, 2024

Motivation and Context

The test coverage workflow needs the PR number to post comments to PRs. However, the workflow_run context doesn't contain the PR number thus we need to save it.

Description

Save the PR number in a file and upload it as artifact for the second workflow to use.

Contribution Checklist

@TaoChenOSU TaoChenOSU added the python Pull requests for the Python Semantic Kernel label Dec 12, 2024
@TaoChenOSU TaoChenOSU self-assigned this Dec 12, 2024
@github-actions github-actions bot changed the title Add PR number to test coverage workflow Python: Add PR number to test coverage workflow Dec 12, 2024
@markwallace-microsoft markwallace-microsoft removed the python Pull requests for the Python Semantic Kernel label Dec 12, 2024
@TaoChenOSU TaoChenOSU enabled auto-merge December 13, 2024 01:29
@TaoChenOSU TaoChenOSU added this pull request to the merge queue Dec 13, 2024
Merged via the queue into microsoft:main with commit fbbd444 Dec 13, 2024
15 checks passed
@TaoChenOSU TaoChenOSU deleted the taochen/add-pr-number-to-test-ceverage-flow branch December 13, 2024 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Sprint: Done
Development

Successfully merging this pull request may close these issues.

4 participants