Skip to content

I think I added respect for the deactivate command.#14

Closed
Reaper176 wants to merge 1 commit intoSpaceShaman:masterfrom
Reaper176:master
Closed

I think I added respect for the deactivate command.#14
Reaper176 wants to merge 1 commit intoSpaceShaman:masterfrom
Reaper176:master

Conversation

@Reaper176
Copy link
Copy Markdown

This is messy but it works and I have so far not managed to break it.

@Reaper176 Reaper176 marked this pull request as draft February 11, 2026 23:35
@Reaper176 Reaper176 marked this pull request as ready for review February 11, 2026 23:35
@SpaceShaman
Copy link
Copy Markdown
Owner

Good code is not the one that works, but the one that is easy to read and modify.
Try to refactor your changes. Instead of creating very extensive comments, it is better to divide the code into smaller functions that describe themselves, it is much more readable and easy to modify later.
https://en.wikipedia.org/wiki/Self-documenting_code

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