Skip to content

Conversation

jizusun
Copy link
Contributor

@jizusun jizusun commented Nov 5, 2018

Copy link
Collaborator

@wolframkriesing wolframkriesing left a comment

Choose a reason for hiding this comment

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

Thanks a lot for the good ideas. I hope my amendments make sense. Would love to hear feedback from you.

assert.deepEqual(arr, [8,8,8]);
});

it('when gets passed an object, it will copy the reference and fill the array with references to that object.', function(){
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
it('when gets passed an object, it will copy the reference and fill the array with references to that object.', function(){
it('when passing an object, it will copy the reference and fill the array with references to that object', function(){

just a picky change :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually, I just copied the whole sentence from MDN 😅

When the fill method gets passed an object, it will copy the reference and fill the array with references to that object.

I'm not confident at all, but it feel like that it should be a passive voice.

arr[0].hi = "es6 katas";

assert.deepEqual(arr[2].hi, 'es6 katas');
});
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't understand the test description and how it is supposed to make the user arrive at the solution.
What I understand from the test description would be the following code (passing test).

const obj = {};
const arr = Array(1).fill(obj);
assert.equal(arr[0], obj);

wouldn't it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, you're right 😂

@jizusun
Copy link
Contributor Author

jizusun commented Nov 22, 2018

Hi sir, is it better now?

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