Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Use a single synapse for all unread tests. #11489

Merged
merged 1 commit into from
Aug 30, 2023
Merged

Use a single synapse for all unread tests. #11489

merged 1 commit into from
Aug 30, 2023

Conversation

andybalaam
Copy link
Member

@andybalaam andybalaam commented Aug 30, 2023 •
edited by github-actions bot
Loading

Saves about 30 seconds per run.

Part of element-hq/element-web#25449

This change is marked as an internal change (Task), so will not be included in the changelog.

@andybalaam
Use a single synapse for all unread tests.
556f111 
Saves about 30 seconds per run.
@andybalaam andybalaam added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Aug 30, 2023
@andybalaam andybalaam requested a review from a team as a code owner August 30, 2023 10:55
germain-gg
germain-gg approved these changes Aug 30, 2023
View reviewed changes
Copy link
Contributor

@germain-gg germain-gg left a comment

Choose a reason for hiding this comment

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

LGTM :shipit:

@andybalaam andybalaam added this pull request to the merge queue Aug 30, 2023
Merged via the queue into develop with commit aaea42c Aug 30, 2023
24 checks passed
@andybalaam andybalaam deleted the andybalaam/single-synpse-for-unread-tests branch August 30, 2023 15:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@germain-gg germain-gg germain-gg approved these changes

@robintown robintown Awaiting requested review from robintown robintown is a code owner automatically assigned from matrix-org/element-web-reviewers

Assignees
No one assigned
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@andybalaam @germain-gg