Added Perl Seed Script ! issue #4 - #26
Conversation
|
Good work! 👍 Looks like the build is failing because you need to run I'll give it a merge once the tests pass. |
| } | ||
|
|
||
| sub plant_a_seed { | ||
| my $seed = shift; |
There was a problem hiding this comment.
Can you use a named argument rather than shift? You defined $type up on line 10 but never used it.
There was a problem hiding this comment.
In fact "shift" get the first argument pass to function call.
At line 21 there is main program with function call and use of $type.
Write your code below
if ($water){
water_the_plant(${type}) ;
}
else {
plant_a_seed(${type});
}
But i understand why you don't find
|
|
||
| =head1 SYNOPSIS | ||
| sample [options] [file ...] | ||
| Options: |
There was a problem hiding this comment.
You may need to include the word usage in the output here to get the tests to pass.
There was a problem hiding this comment.
ok thank,
I add it very soon.
regards
| <option value="python">Python</option> | ||
| <option value="ruby">Ruby</option> | ||
| <option value="scala">Scala</option> | ||
| <option value="perl">Perl</option> |
There was a problem hiding this comment.
These lists are in alphabetical order. Not a big deal - I'll merge it and rearrange them.
| # things if we do some refactoring. | ||
|
|
||
| DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null && pwd )" | ||
| chmod +x $DIR/seeds/*.* |
There was a problem hiding this comment.
Rather than doing this, I've just been checking all the scripts into Git with the x bit set. Again, not a big deal.
Hi
Please find the Perl Seed Script.
Say me if it is Ok for you.
regards