-
Notifications
You must be signed in to change notification settings - Fork 12.5k
Fixed few examples #64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
It seems there are some more on these lines: 123 const match = cityStateRegex.match(cityStateRegex) // < HERE209 var breweryName = name || 'Hipster Brew Co.' // < HERE245 var menuConfig = {
title: 'Foo',
body: 'Bar',
buttonText: 'Baz',
cancellable: true
} // < HERE482 var menuConfig = {
title: null,
body: 'Bar',
buttonText: null,
cancellable: true
} // < HERE502 var menuConfig = {
title: 'Order',
// User did not include 'body' key
buttonText: 'Send',
cancellable: true
} // < HERE584 var name = 'Ryan McDermott' // < HERE982 var Employee = function(name) {
this.name = name;
} // < HERE986 Employee.prototype.getName = function() {
return this.name;
} // < HERE1735 + 1737 require('request').get('https://en.wikipedia.org/wiki/Robert_Cecil_Martin', function(err, response) {
if (err) {
console.error(err);
}
else {
require('fs').writeFile('article.html', response.body, function(err) {
if (err) {
console.error(err);
} else {
console.log('File written');
}
}) // < HERE
}
}) // < HERE1752 require('request-promise').get('https://en.wikipedia.org/wiki/Robert_Cecil_Martin')
.then(function(response) {
return require('fs-promise').writeFile('article.html', response);
})
.then(function() {
console.log('File written');
})
.catch(function(err) {
console.error(err);
}) // < HERE1775 require('request-promise').get('https://en.wikipedia.org/wiki/Robert_Cecil_Martin')
.then(function(response) {
return require('fs-promise').writeFile('article.html', response);
})
.then(function() {
console.log('File written');
})
.catch(function(err) {
console.error(err);
}) // < HERE1783 var request = await require('request-promise') // < HERE2107 let actions = function() {
// ...
} // < HERE2119 let actions = function() {
// ...
} // < HERE |
|
@vsemozhetbyt Great catch!!! I have added above changes. |
|
Awesome finds! Can you rebase this with master and resolve conflicts? There was a large change after #51 which has affected this branch a little bit. Thank you! |
|
@ryanmcdermott done :) |
|
@HemantPawar Just found one more :) Line 597: Array.prototype.diff = function(comparisonArray) {
const values = [];
const hash = {};
for (let i of comparisonArray) {
hash[i] = true;
}
for (let i of this) {
if (!hash[i]) {
values.push(i);
}
}
return values;
} // < HERE |
|
And the line 1514: car.setModel('F-150') // < HERE |
|
Thanks @vsemozhetbyt :) |
|
@ryanmcdermott resolved conflicts |
|
Thanks Hemant! While these changes may feel small, they go such a long way to improving the readability of this project 😄 |
Fixed few lines by adding missing semicolons.