Skip to content

Conversation

@kojix2
Copy link
Member

@kojix2 kojix2 commented May 11, 2019

No description provided.

IRuby.logger.level = @args.delete('--debug') ? Logger::DEBUG : Logger::INFO

raise(ArgumentError, 'Not enough arguments to the kernel') if @args.size < 2 || @args.size > 4
raise(ArgumentError, 'Not enough arguments to the kernel') unless @args.size == 3
Copy link
Contributor

@mrkn mrkn May 13, 2019

Choose a reason for hiding this comment

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

Could you please change the message to Wrong number of arguments to the kernel?

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed. Thanks!

@kojix2 kojix2 closed this May 23, 2019
@kojix2 kojix2 deleted the kojix2-patch branch May 23, 2019 11:51
@kojix2
Copy link
Member Author

kojix2 commented May 23, 2019

if a < 2 || a > 4
mean
(2..4).include? a

That was my fault

Yuki-Inoue referenced this pull request in Yuki-Inoue/iruby Jan 3, 2021
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