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

Issue #34: Fixed minor spelling & grammar mistakes for clarity #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DevOpsJeremy
Copy link

@DevOpsJeremy DevOpsJeremy commented Jan 30, 2024

This fixes minor spelling/grammar mistakes in the README in defined in this issue and has no effect on the functionality.

Line 5

Previous

There is no throttling mechanism and so all jobs are started immediately and are all run currently.

The intended word here is 'concurrently'. The distinction is important because concurrent operations is the purpose of this module.

Current

There is no throttling mechanism and so all jobs are started immediately and are all run concurrently.

Line 9

Previous

ThreadJob jobs will tend to run much faster because there is lower overhead and they do not use the remoting serialization system as BackgroundJob jobs do. And they will use up fewer system resources.

This snippet was reworded for clarity and to fix a minor issue with parallel structure.

Current

ThreadJob jobs will tend to run much faster because there is lower overhead and they do not use the remoting serialization system as BackgroundJob jobs do. Additionally, they consume fewer system resources.

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