Skip to content

Remove fastrand dependency#61

Merged
james7132 merged 1 commit into
smol-rs:masterfrom
james7132:remove-fastrand-dependency
Apr 21, 2024
Merged

Remove fastrand dependency#61
james7132 merged 1 commit into
smol-rs:masterfrom
james7132:remove-fastrand-dependency

Conversation

@james7132
Copy link
Copy Markdown
Contributor

fastrand isn't being used anywhere in the crate. This PR removes it as a required dependency.

Copy link
Copy Markdown
Contributor

@notgull notgull left a comment

Choose a reason for hiding this comment

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

Thanks!

fastrand was originally used by the pipe implementation back when it was inlined into this crate and not used by piper.

@notgull
Copy link
Copy Markdown
Contributor

notgull commented Apr 21, 2024

@james7132 Thank you for all of your contributions to smol! I have sent you an invite to the smol-rs organization. If you accept, you should be able to merge this PR.

@james7132 james7132 merged commit f6fea36 into smol-rs:master Apr 21, 2024
@notgull notgull mentioned this pull request Apr 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants