Skip to content

Conversation

@zydan1
Copy link

@zydan1 zydan1 commented Jan 12, 2018

hello teacher Could you please accept my pull request

Copy link
Owner

@remarcmij remarcmij left a comment

Choose a reason for hiding this comment

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

Almost there! There is still one bug: I still get a Yum when the robot has moved beyond the apples and when reaching the flag. I should only get Yum's when passing over an apple.

The pull request itself has now the correct branch.

@@ -1,21 +1,22 @@
(function () {
((function () {
Copy link
Owner

Choose a reason for hiding this comment

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

You don't need these extra parentheses. As I explained earlier this is sufficient:

(function () {
    // all your other code
})();

y: 0,
dir: 'up',
y: 2,
dir: 'down',
Copy link
Owner

Choose a reason for hiding this comment

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

The initial direction of the robot in the example is 'right'.

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