Skip to content

Conversation

MahBoiDeveloper
Copy link
Contributor

No description provided.

Copy link

github-actions bot commented Oct 3, 2025

Nightly build for this pull request:

  • artifacts.zip
    This comment is automatic and is meant to allow guests to get latest automatic builds without registering. It is updated on every successful build.

@SadPencil
Copy link
Member

This class contains no extension method. So why moving it to the Extensions namespace?

@MahBoiDeveloper
Copy link
Contributor Author

This class contains no extension method. So why moving it to the Extensions namespace?

Because it extends the functionality of the File class even if have no extension method like void DoSomething(this File file) {}. Previous naming was a bit incorrect.

@SadPencil
Copy link
Member

This class contains no extension method. So why moving it to the Extensions namespace?

Because it extends the functionality of the File class even if have no extension method like void DoSomething(this File file) {}. Previous naming was a bit incorrect.

We have 4 classes whose name ends up with Helper now. Can you further explain why the helper name is incorrect?

@MahBoiDeveloper
Copy link
Contributor Author

We have 4 classes whose name ends up with Helper now. Can you further explain why the helper name is incorrect?

Helper classes are mostly incorrectly named because they are extensions, managers, inbetween entities.

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