Skip to content

Add Etc signature #1006

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

Merged
merged 1 commit into from
May 25, 2022
Merged

Add Etc signature #1006

merged 1 commit into from
May 25, 2022

Conversation

ybiquitous
Copy link
Contributor

The base of this change is generated by the following steps:

  1. Run RUBYOPT=-retc bundle exec exe/rbs prototype runtime --merge Etc > stdlib/etc/0/etc.rbs
  2. Run RUBYOPT=-retc bundle exec exe/rbs prototype runtime --merge Etc::Group >> stdlib/etc/0/etc.rbs
  3. Run RUBYOPT=-retc bundle exec exe/rbs prototype runtime --merge Etc::Passwd >> stdlib/etc/0/etc.rbs
  4. Run bundle exec exe/rbs annotate stdlib/etc/0/*.rbs
  5. Run bundle exec rake 'generate:stdlib_test[Etc]'

Then, I've edited the files manually.

See https://docs.ruby-lang.org/en/3.1/Etc.html

Note: It's been a while since I've written, so feel free to point out any style violations, insufficient test cases, etc.

@ybiquitous ybiquitous marked this pull request as draft May 23, 2022 01:32
@ybiquitous ybiquitous force-pushed the add-etc-module-signature branch from be26fe7 to 16e787b Compare May 23, 2022 01:33
@ybiquitous ybiquitous marked this pull request as ready for review May 23, 2022 01:38
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.

🐱

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.

Thanks! 👍

The base of this change is generated by the following steps:

1. Run `RUBYOPT=-retc bundle exec exe/rbs prototype runtime --merge Etc > stdlib/etc/0/etc.rbs`
2. Run `RUBYOPT=-retc bundle exec exe/rbs prototype runtime --merge Etc::Group >> stdlib/etc/0/etc.rbs`
3. Run `RUBYOPT=-retc bundle exec exe/rbs prototype runtime --merge Etc::Passwd >> stdlib/etc/0/etc.rbs`
4. Run `bundle exec exe/rbs annotate stdlib/etc/0/*.rbs`
5. Run `bundle exec rake 'generate:stdlib_test[Etc]'`

Then, I've edited the files manually.

See https://docs.ruby-lang.org/en/3.1/Etc.html
@ybiquitous ybiquitous force-pushed the add-etc-module-signature branch from 186b294 to 012c49e Compare May 24, 2022 00:16
@ybiquitous
Copy link
Contributor Author

@soutaro Thanks for the review. I've put it in one commit.

@soutaro soutaro merged commit 4dc5b95 into ruby:master May 25, 2022
@ybiquitous ybiquitous deleted the add-etc-module-signature branch May 25, 2022 13:04
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