Skip to content

Conversation

etpinard
Copy link
Collaborator

No description provided.

same as dash.py uses
@etpinard
Copy link
Collaborator Author

etpinard commented Jul 19, 2024

eb02e2a makes the build workflow pass again.

Getting Percy back to 🟢 will require more attention.

@etpinard
Copy link
Collaborator Author

The percy problems might be related to: plotly/dash#2747

@etpinard
Copy link
Collaborator Author

Confirmed.

We pull in the latest dev dash.py changes on CircleCI here:

git clone --depth 1 https://github.com/plotly/dash.git -b dev dash-main

meaning the changes in dash.testing from plotly/dash#2747 we're pulled in.

Inspecting the changes in https://percy.io/plotly/dash.jl/builds/35406873 we see that only the filenames changed. The baselines are identical otherwise.

Unfortunately, looks like I am no longer able to approve changes up on Percy:

Screencast.from.2024-07-19.14-43-42.mp4

(the Approve build button is disabled)

Can someone at Plotly (cc'ing https://github.com/gvwilson) approve these changes? Thank you!

@etpinard
Copy link
Collaborator Author

ping @gvwilson

@gvwilson gvwilson self-requested a review September 20, 2024 18:06
@etpinard etpinard merged commit e98c6eb into dev Sep 20, 2024
@etpinard etpinard deleted the fix-circleci branch September 20, 2024 18:33
@etpinard etpinard mentioned this pull request Sep 11, 2025
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.

2 participants