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

15805 version in azure events #15902

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

Conversation

jack-h-wang
Copy link
Collaborator

@jack-h-wang jack-h-wang commented Sep 17, 2024

This PR makes the current commit hash available to the main project and adds it to Azure events.

Test Steps:

  1. Run Gradle task package or quickPackage and ensure the project builds successfully.

Changes

  • Add Gradle task to generate a Version object usable in the main project.
  • Add version to ReportEventData and supply the commit hash from the Version object.

Checklist

Testing

  • Tested locally?
  • Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
  • Added tests?

Linked Issues

@jack-h-wang jack-h-wang added the platform Platform Team label Sep 17, 2024
Copy link

github-actions bot commented Sep 17, 2024

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

Copy link

github-actions bot commented Sep 17, 2024

Test Results

1 229 tests  ±0   1 225 ✅ ±0   7m 0s ⏱️ -38s
  161 suites ±0       4 💤 ±0 
  161 files   ±0       0 ❌ ±0 

Results for commit bd85a2e. ± Comparison against base commit 675b0e1.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Sep 17, 2024

Integration Test Results

 53 files  ±0   53 suites  ±0   28m 13s ⏱️ +34s
410 tests ±0  401 ✅ ±0  9 💤 ±0  0 ❌ ±0 
413 runs  ±0  404 ✅ ±0  9 💤 ±0  0 ❌ ±0 

Results for commit bd85a2e. ± Comparison against base commit 675b0e1.

♻️ This comment has been updated with latest results.

Copy link

sonarcloud bot commented Sep 20, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform Platform Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant