Skip to content

perf: Replace getById call with getFirstNodeById#2360

Open
CarlSchwan wants to merge 2 commits intomasterfrom
carl/perf-getbyid
Open

perf: Replace getById call with getFirstNodeById#2360
CarlSchwan wants to merge 2 commits intomasterfrom
carl/perf-getbyid

Conversation

@CarlSchwan
Copy link
Member

And some other minor cleanup in ViewInfoCache

And some other minor cleanup in ViewInfoCache

Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan CarlSchwan added this to the Nextcloud 34 milestone Feb 5, 2026
@CarlSchwan CarlSchwan self-assigned this Feb 5, 2026
Fix psalm when running locally

Signed-off-by: Carl Schwan <carlschwan@kde.org>
@nickvergessen nickvergessen requested review from miaulalala and removed request for nickvergessen February 5, 2026 15:39
@cypress
Copy link

cypress bot commented Feb 5, 2026

Activity    Run #3289

Run Properties:  status check passed Passed #3289  •  git commit 0ca180ab46: perf: Replace getById call with getFirstNodeById
Project Activity
Branch Review carl/perf-getbyid
Run status status check passed Passed #3289
Run duration 02m 21s
Commit git commit 0ca180ab46: perf: Replace getById call with getFirstNodeById
Committer Carl Schwan
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 1
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 9
View all changes introduced in this branch ↗︎

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.

1 participant