Skip to content

Added Concurrency#20

Merged
DavidSalazar123 merged 3 commits intomainfrom
test
Feb 23, 2024
Merged

Added Concurrency#20
DavidSalazar123 merged 3 commits intomainfrom
test

Conversation

@DavidSalazar123
Copy link
Copy Markdown
Collaborator

@DavidSalazar123 DavidSalazar123 commented Feb 22, 2024 •
edited
Loading

  • Added concurrency to allow multiple servers be notified of jobs
  • Added NoSQL database to track all channels with bot
  • Added locks to prevent accidentally sending jobs to wrong servers
  • Changed dictionary to sets within InternshipUtilities.py
  • Added .toml to have consistent formatting

aarontorres0
aarontorres0 previously approved these changes Feb 23, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@aarontorres0 aarontorres0 left a comment

Choose a reason for hiding this comment

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

lgtm

@DavidSalazar123 DavidSalazar123 merged commit 879f5cb into main Feb 23, 2024
@DavidSalazar123 DavidSalazar123 deleted the test branch February 23, 2024 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aarontorres0 aarontorres0 aarontorres0 left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@DavidSalazar123 @aarontorres0