Skip to content

/rust-threadpool

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 keywords and categories to Cargo.toml. #88

Merged
merged 1 commit into from Aug 24, 2017

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@mjkillough @KodrAus @frewsxcv
@mjkillough
Copy link
Contributor

mjkillough commented Aug 22, 2017

  • Add a set of keywords that I'd search for when looking for a similar
    library.
  • Add the 'os' category - there seem to be other libraries in that
    category that offer similar abstractions on top of OS level features
    like threads.
  • Considered (but didn't) add the 'asynchronous' category. The
    crates.io description for that category suggest it's exclusively for
    Tokio-like libraries.

Fixes #84.
@mjkillough
Add keywords and categories to Cargo.toml. 
 - Add a set of keywords that I'd search for when looking for a similar
   library.
 - Add the 'os' category - there seem to be other libraries in that
   category that offer similar abstractions on top of OS level features
   like threads.
 - Considered (but didn't) add the 'asynchronous' category. The
   crates.io description for that category suggest it's exclusively for
   Tokio-like libraries.

Fixes #84.
Loading status checks…
841e60e
@KodrAus

This comment has been minimized.

Sign in to view
Copy link

KodrAus commented Aug 23, 2017

Thanks @mjkillough! This looks good to me. Are you happy with those categories @frewsxcv?
@frewsxcv

This comment has been minimized.

Sign in to view
Copy link
Collaborator

frewsxcv commented Aug 24, 2017

LGTM, thanks!

@frewsxcv frewsxcv merged commit 680ff6a into rust-threadpool:master Aug 24, 2017
1 check passed

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.