Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consistent naming of interfaces and their implementations #1421

Open
heiko-holz opened this issue May 5, 2022 · 0 comments
Open

Consistent naming of interfaces and their implementations #1421

heiko-holz opened this issue May 5, 2022 · 0 comments
Labels
area/general enhancement New feature or request invalid This doesn't seem right

Comments

@heiko-holz
Copy link
Contributor

Currently, the project inconsistently uses the I prefix in interfaces.

E.g., public interface IGenericAction and public interface CommandExecutor.

We should adhere to a naming convention.

In this ticket, a naming convention for interfaces should be established and the existing interfaces and implementations should be refactored accordingly.

Things to consider

  • naming of the interfaces
    • use the I prefix or not?
  • naming of the implementations
    • when to use the Impl suffix in the implementations?
@heiko-holz heiko-holz added enhancement New feature or request invalid This doesn't seem right labels May 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/general enhancement New feature or request invalid This doesn't seem right
Projects
Status: No status
Development

No branches or pull requests

2 participants