Skip to content

Add workflow step that shows build scan url#11088

Merged
trask merged 5 commits into
open-telemetry:mainfrom
laurit:build-scan
Apr 11, 2024
Merged

Add workflow step that shows build scan url#11088
trask merged 5 commits into
open-telemetry:mainfrom
laurit:build-scan

Conversation

@laurit

@laurit laurit commented Apr 10, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@laurit laurit requested a review from a team April 10, 2024 11:50

@zeitlinger zeitlinger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

great!

Comment thread .github/workflows/build-common.yml Outdated
Comment on lines +272 to +276
if: ${{ hashFiles('build-scan.txt') != '' }}
run: echo "build-scan=$(cat build-scan.txt)" >> $GITHUB_OUTPUT
- name: Build scan
if: ${{ hashFiles('build-scan.txt') != '' }}
run: echo ${{ steps.read-build-scan.outputs.build-scan }}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why 2 steps instead of 1?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The sample code I found from google used 2 steps. I was intending on relying on you pointing out a simpler solution if one exists.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

maybe the $GITHUB_OUTPUT is displayed in some summary box?

Comment thread .github/workflows/build-common.yml Outdated
Comment thread .github/workflows/reusable-test-indy.yml Outdated
Comment thread .github/workflows/reusable-test-latest-deps.yml Outdated
Comment thread .github/workflows/reusable-test-latest-deps.yml
@trask trask merged commit 4c77b48 into open-telemetry:main Apr 11, 2024
@tylerbenson

Copy link
Copy Markdown
Member

Thanks for adding this @laurit!

@laurit laurit deleted the build-scan branch May 24, 2024 07:24
laurit added a commit to laurit/opentelemetry-java-instrumentation that referenced this pull request May 24, 2024
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
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.

5 participants