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

Update event schema version type #421

Merged
merged 4 commits into from
Dec 18, 2024

Conversation

davidbrochart
Copy link
Collaborator

No description provided.

Copy link
Contributor

Binder 👈 Launch a Binder on branch davidbrochart/jupyter_collaboration/event-version

@davidbrochart
Copy link
Collaborator Author

It looks like UI tests are flaky again @trungleduc?
For instance:

  • actual:
    image
  • expected:
    image
  • diff:
    image

@trungleduc
Copy link
Member

yeah, your only choice is to relax the diff ratio

@@ -20,7 +20,7 @@ module.exports = {
},
expect: {
toMatchSnapshot: {
maxDiffPixelRatio: 0.0001
maxDiffPixelRatio: 0.01
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
maxDiffPixelRatio: 0.01
maxDiffPixelRatio: 0.001

1% might be overkill

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It was not, I'm reverting that change.

@davidbrochart
Copy link
Collaborator Author

Ubuntu test failures are tracked in #416.

@krassowski
Copy link
Member

Ubuntu test failures are tracked in #416.

Should be fixed in 15 minutes with release of 4.3.4 🤞

@davidbrochart
Copy link
Collaborator Author

Oh, let's wait for it then 👍

@davidbrochart
Copy link
Collaborator Author

Let's see if it picks up JupyterLab v4.3.4.

@davidbrochart davidbrochart reopened this Dec 18, 2024
@davidbrochart
Copy link
Collaborator Author

CI is green again 🎉

@davidbrochart davidbrochart merged commit 0a75715 into jupyterlab:main Dec 18, 2024
47 of 52 checks passed
@davidbrochart davidbrochart deleted the event-version branch December 18, 2024 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants