-
Notifications
You must be signed in to change notification settings - Fork 367
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
chore: fix the CI benchmarks posting and also exit gracefully if not …
…even relevant (#6966) * chore: actually add deltas to the artifact as this changed from v3-v4 * chore: add conditions and check for file existence
- Loading branch information
1 parent
d45e450
commit 5552659
Showing
2 changed files
with
9 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -18,12 +18,19 @@ jobs: | |
workflow: ${{ github.event.workflow.id }} | ||
workflow_conclusion: success | ||
name: delta-action-deltas | ||
if_no_artifact_found: ignore | ||
github_token: ${{ secrets.GITHUB_TOKEN }} | ||
- name: Check File Existence | ||
id: check-file-existence | ||
uses: thebinaryfelix/[email protected] | ||
with: | ||
files: '.delta.*' | ||
- name: Get PR number | ||
if: github.event.workflow_run.event == 'pull_request' | ||
if: github.event.workflow_run.event == 'pull_request' && steps.check-file-existence.outputs.exists == 'true' | ||
id: pr_number | ||
run: echo "pr_number=$(cat pr_number)" >> $GITHUB_OUTPUT | ||
- name: Post deltas to GitHub | ||
if: steps.check-file-existence.outputs.exists == 'true' | ||
uses: netlify/delta-action@v4 | ||
with: | ||
title: '📊 Benchmark results' | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -37,6 +37,7 @@ jobs: | |
with: | ||
name: delta-action-deltas | ||
retention-days: 7 | ||
include-hidden-files: true | ||
path: | | ||
.delta.* | ||
pr_number |
5552659
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📊 Benchmark results