Skip to content

Conversation

@jrjohnson
Copy link
Member

Found a way nicer actions based way to create these comments and then attach a marker to them so if we re-run a PR we can replace the comment instead of creating it again.

Found a way nicer actions based way to create these comments and then
attach a marker to them so if we re-run a PR we can replace the
comment instead of creating it again.
@netlify
Copy link

netlify bot commented Jan 29, 2026

Deploy Preview for ilios-frontend ready!

Name Link
🔨 Latest commit 70542bf
🔍 Latest deploy log https://app.netlify.com/projects/ilios-frontend/deploys/697ae0fd5323330008cc0e40
😎 Deploy Preview https://deploy-preview-9104--ilios-frontend.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link
Contributor

🔍 Visual Diff Results

🔍 Comparing screenshots...

Baseline: /home/runner/work/frontend/frontend/baseline-frontend
Candidate: /home/runner/work/frontend/frontend/candidate-frontend
Output: /home/runner/work/frontend/frontend/visual-diff-9104

📊 Results:
Total images: 752
With differences: 0
Identical: 752
Removed: 0
Added: 0

✅ All checks passed!

📄 Report generated: /home/runner/work/frontend/frontend/visual-diff-9104/index.html

Download the results.

@jrjohnson jrjohnson marked this pull request as ready for review January 29, 2026 05:21
run: |
JSON='${{ steps.data.outputs.result }}'

ISSUE_NUMBER="$(echo "$JSON" | jq -r '.issue_number')"
Copy link
Contributor

Choose a reason for hiding this comment

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

You can use jq in a github workflow script? I didn't think that was standard like awk, sed, etc.

Copy link
Member Author

@jrjohnson jrjohnson Jan 29, 2026

Choose a reason for hiding this comment

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

I kind of thought of it that way. Always seems to be there when I want it. I'm this case it's one of the installed tools: https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2404-Readme.md#tools

actions: read
pull-requests: write
env:
MARKER: "<!-- visual-diff-results -->"
Copy link
Member

Choose a reason for hiding this comment

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

i accept this.

@dartajax dartajax added the run ui tests Run the expensive UI tests label Jan 31, 2026
@jrjohnson jrjohnson merged commit d93645c into ilios:master Jan 31, 2026
47 checks passed
@jrjohnson jrjohnson deleted the replace-diff-comment branch January 31, 2026 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run ui tests Run the expensive UI tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants