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

Updated PublishTestResults.zip artifact for PublishTestResultsV2 #20899

Merged
merged 3 commits into from
Mar 5, 2025

Conversation

virekumar
Copy link
Contributor

@virekumar virekumar commented Mar 3, 2025

Task name: PublishTestResultsV2

Description: Updated PublishTestResults.zip artifact for PublishTestResultsV2.
Added customTests functionality to NUnit2.5 as well.

image

image

Risk Assesment(Low/Medium/High): Low

Added unit tests: (Y/N) : Already added in mseng repo. Not required here

Tests Performed: Manually tested Nunit2.5 file upload

Documentation changes required: N/A

Attached related issue: N/A

Checklist:

  • Task version was bumped - please check instruction how to do it
  • Checked that applied changes work as expected

@virekumar
Copy link
Contributor Author

/azp run

1 similar comment
@virekumar
Copy link
Contributor Author

/azp run

@virekumar virekumar enabled auto-merge (squash) March 4, 2025 05:28
@virekumar
Copy link
Contributor Author

/azp run

Copy link
Contributor

@adityashahms adityashahms left a comment

Choose a reason for hiding this comment

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

LGTM

@virekumar virekumar disabled auto-merge March 5, 2025 06:01
@virekumar
Copy link
Contributor Author

/azp run

@virekumar
Copy link
Contributor Author

/azp run

@rishabhmalikMS rishabhmalikMS merged commit fa63835 into master Mar 5, 2025
5 checks passed
@virekumar virekumar deleted the users/virekumar/publish-new-artifact0ptr2 branch March 5, 2025 06:45
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