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

Add documentation regarding thread creation logic #36

Merged
merged 5 commits into from
Mar 2, 2023

Conversation

benfoster04
Copy link
Collaborator

Based off information in #30.

Based off information in #30.

Signed-off-by: Ben Foster <[email protected]>
@benfoster04 benfoster04 self-assigned this Mar 2, 2023
@benfoster04 benfoster04 added the Type: Documentation Improvements or additions to documentation. label Mar 2, 2023
Copy link
Collaborator

@kk5dire kk5dire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

main_utils.py Outdated Show resolved Hide resolved
main_utils.py Outdated Show resolved Hide resolved
@benfoster04 benfoster04 requested a review from Tech-TTGames March 2, 2023 19:22
Copy link
Owner

@Tech-TTGames Tech-TTGames left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@benfoster04 benfoster04 merged commit 2209ab5 into dev Mar 2, 2023
@benfoster04 benfoster04 deleted the benfoster04-patch-episode2 branch March 2, 2023 19:29
@Tech-TTGames Tech-TTGames added Status: Completed It's done! The feature/fix has been deployed. Archive: Old Style This issue has been submitted before a major reform in issue style. It cannot be used as reference. labels Mar 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Archive: Old Style This issue has been submitted before a major reform in issue style. It cannot be used as reference. Status: Completed It's done! The feature/fix has been deployed. Type: Documentation Improvements or additions to documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants