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 README.md #624

Merged
merged 2 commits into from
Oct 30, 2024
Merged

Update README.md #624

merged 2 commits into from
Oct 30, 2024

Conversation

jluethi
Copy link
Collaborator

@jluethi jluethi commented Oct 30, 2024

No description provided.

@jluethi
Copy link
Collaborator Author

jluethi commented Oct 30, 2024

@zonia3000 Any idea why we run over the timeout in the Node20 tests here? Have you observed this before? I assume it shouldn't be related to the addition of the icons to the doc page & the README.

The error I see in the github action for the end-to-end tests: Test timeout of 30000ms exceeded

@zonia3000
Copy link
Collaborator

I've rerun the job and it passed. We have some sporadic failures very difficult to reproduce locally. I added some mitigations in the past, but it is still not perfect.

Sometimes it might be caused by backend issues happening with high concurrency, as discussed here: #558 (comment)

@jluethi
Copy link
Collaborator Author

jluethi commented Oct 30, 2024

Great, thanks. I'll merge then

@jluethi jluethi merged commit 1a42138 into main Oct 30, 2024
7 checks passed
@jluethi jluethi deleted the logo branch October 30, 2024 16:14
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