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

Api docs #81

Merged
merged 9 commits into from
Apr 22, 2024
Merged

Api docs #81

merged 9 commits into from
Apr 22, 2024

Conversation

BoolPurist
Copy link
Collaborator

No description provided.

Removed unused usage of workspace dependencies in "usermgmt_gui"
Detected unused dependencies with cargo tool "cargo-machete"
Unwraps were replaced with expects or proper error propagation
Made some structs read-only
Found those errors via CLI tool typos:
https://github.com/crate-ci/typos
Documented CommandBuilder
Moved group struct into own module
Removed redundancy in its key loop ups
Refactored: Mobilized types into own modules
Refactored: Bundled 4 callbacks into own trait to shorten argument
count in some functions
@BoolPurist
Copy link
Collaborator Author

Relates to issue #74

@BoolPurist BoolPurist marked this pull request as ready for review April 22, 2024 21:07
@BoolPurist BoolPurist merged commit 1d78342 into th-nuernberg:main Apr 22, 2024
1 check passed
@BoolPurist BoolPurist deleted the api_docs branch May 4, 2024 09:26
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.

1 participant