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

[openssl] Add RBS file for "openssl" #743

Merged
merged 25 commits into from
Aug 18, 2021
Merged

Conversation

HoneyryderChuck
Copy link
Contributor

Added RBS signatures for the "openssl" package

Also adds the following:

  • Improve IO.select signature for more accurate return type without timeout;
  • Improve SecureRandom.random_number signature for more accurate return type based on argument;
  • Added exception: false option support for IO#read_nonblock/write_nonblock;

@HoneyryderChuck
Copy link
Contributor Author

@soutaro sider checks fail in something I find debatable, i.e. block arguments shouldn't have the same expectation for boolish (at least in these cases, we know a bool will be yielded to block).

Copy link
Member

@soutaro soutaro left a comment

Choose a reason for hiding this comment

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

👍

@soutaro soutaro merged commit 3843349 into ruby:master Aug 18, 2021
@HoneyryderChuck HoneyryderChuck deleted the openssl branch August 18, 2021 13:50
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