Skip to content

Conversation

@skjnldsv
Copy link
Member

We keep adding more and more tests. It's been a while since we increased.

Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
@skjnldsv skjnldsv self-assigned this Aug 20, 2025
@skjnldsv skjnldsv requested a review from a team as a code owner August 20, 2025 14:57
@skjnldsv skjnldsv requested review from Altahrim, come-nc and salmart-dev and removed request for a team August 20, 2025 14:57
@skjnldsv skjnldsv requested review from a team, nfebe, susnux and szaimen and removed request for a team August 20, 2025 14:57
@skjnldsv skjnldsv enabled auto-merge August 20, 2025 14:57
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

I wonder, why do we have a limit at all if it speeds things up?

@skjnldsv
Copy link
Member Author

I wonder, why do we have a limit at all if it speeds things up?

Because at some point it's slower to set one runner up than run x amount of tests.
There's a fine line :)

@szaimen
Copy link
Contributor

szaimen commented Aug 20, 2025

I wonder, why do we have a limit at all if it speeds things up?

Because at some point it's slower to set one runner up than run x amount of tests. There's a fine line :)

I see :)

@szaimen szaimen added this to the Nextcloud 32 milestone Aug 20, 2025
@skjnldsv skjnldsv disabled auto-merge August 20, 2025 15:59
@skjnldsv skjnldsv merged commit 93b97f4 into master Aug 20, 2025
222 of 231 checks passed
@skjnldsv skjnldsv deleted the chore/cypress-runners branch August 20, 2025 15:59
@nextcloud-bot nextcloud-bot mentioned this pull request Aug 22, 2025
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants