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

Adaptation to C# coding convention #215

Open
Rivixer opened this issue May 22, 2023 · 0 comments
Open

Adaptation to C# coding convention #215

Rivixer opened this issue May 22, 2023 · 0 comments
Assignees
Labels
difficulty: 2 Standard task priority: 1 This doesn't have to be done unless there are no other tasks scope: developers This task refers to group of developers type: chore For general maintenance and upkeep tasks

Comments

@Rivixer
Copy link
Owner

Rivixer commented May 22, 2023

We wszystkich klasach, które są "internal", wszystkie rzeczy (pola, właściwości, metody itp.), które są ograniczone "internal", zmienić na "public". Klasa powinna nadal zostać internal.

Przeanalizować wszystkie nazwy pól i upewnić się, że:

  • prywatne pola są napisane camelCasem i poprzedzone "_", np: _privateField
  • statyczne prywatne pola są napisane camelCasel i poprzedzone "s_", np: s_staticPrivateField
  • wszystkie stałe pola są napisane PascalCasem (chyba nie mamy stałych pól w kodzie)

Wszystkie pola publiczne/chronione zamienić na właściwości (co wiąże się z przeniesieniem pod konstruktor, oraz zmiany komentarza - zamiast znacznika "summary" będzie "value")

Przeanalizować tylko WZIMopoly.

To issue może spowodować wiele konfliktów, dlatego najlepiej stworzyć branch tuż przed wzięciem się do roboty i nie zwlekać z jego pull requestowaniem!

@Rivixer Rivixer added scope: developers This task refers to group of developers difficulty: 2 Standard task priority: 1 This doesn't have to be done unless there are no other tasks type: chore For general maintenance and upkeep tasks labels May 22, 2023
This was referenced May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty: 2 Standard task priority: 1 This doesn't have to be done unless there are no other tasks scope: developers This task refers to group of developers type: chore For general maintenance and upkeep tasks
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants