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

Fix CircleCI #239

Merged
merged 1 commit into from
Sep 20, 2024
Merged

Fix CircleCI #239

merged 1 commit into from
Sep 20, 2024

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
12 of 13 checks passed
@etpinard etpinard deleted the fix-circleci branch September 20, 2024 18:33
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