Skip to content

refactor(rector): Run rector on new code#58138

Open
CarlSchwan wants to merge 1 commit intomasterfrom
carl/reactor-run
Open

refactor(rector): Run rector on new code#58138
CarlSchwan wants to merge 1 commit intomasterfrom
carl/reactor-run

Conversation

@CarlSchwan
Copy link
Member

Summary

Run rector on new code

Checklist

@CarlSchwan CarlSchwan added this to the Nextcloud 34 milestone Feb 6, 2026
@CarlSchwan CarlSchwan self-assigned this Feb 6, 2026
@CarlSchwan CarlSchwan added the 3. to review Waiting for reviews label Feb 6, 2026
@CarlSchwan CarlSchwan requested review from ArtificialOwl, come-nc, icewind1991 and provokateurin and removed request for a team February 6, 2026 15:27
@CarlSchwan CarlSchwan force-pushed the carl/reactor-run branch 3 times, most recently from a4f4c20 to 1ced58a Compare February 6, 2026 16:53
Signed-off-by: Carl Schwan <carlschwan@kde.org>

class File extends Node implements \OCP\Files\File {
/**
* Creates a Folder that represents a non-existing path
Copy link
Contributor

Choose a reason for hiding this comment

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

Description is wrong.

Comment on lines -63 to +44
* Creates a Node of the same type that represents a non-existing path
* Get non existing node.
Copy link
Contributor

Choose a reason for hiding this comment

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

Previous description was better 😁

* @throws \Exception
*/
protected function createNonExistingNode($path) {
protected function createNonExistingNode(string $path): Node {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
protected function createNonExistingNode(string $path): Node {
protected function createNonExistingNode(string $path): self {

Should be correct?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants