Skip to content

sandbox exercises - #4

Open
romanV7 wants to merge 1 commit into
HowProgrammingWorks:masterfrom
romanV7:master
Open

sandbox exercises#4
romanV7 wants to merge 1 commit into
HowProgrammingWorks:masterfrom
romanV7:master

Conversation

@romanV7

Copy link
Copy Markdown

I've dove some tasks from READ.md
I had to write " let api " in framework and application files, cause tests didn't wanna pass my code!
But if remove " let api " from these files, code will execute.
As a result, we have to remove " let api " in a bit to make the following code work properly.

@RV8VRV8V left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Change it.

@@ -1,19 +1,47 @@
'use strict';
// Tests requires
let api;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

It really needs to be deleted.

}
};
// Tests require to initialize 'api', but with it, code is not running!
let api;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

And this one too.

Sign up for freeto 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

@romanV7@RV8V