Skip to content
This repository has been archived by the owner. It is now read-only.

Removed arrow functions for Mocha tests. #323

Merged
merged 1 commit into from Apr 1, 2016
Merged

Removed arrow functions for Mocha tests. #323

merged 1 commit into from Apr 1, 2016

Conversation

@hwillson
Copy link
Contributor

@hwillson hwillson commented Apr 1, 2016

Hi guys - this relates to #318 (removing arrow functions for Mocha tests and updating the sample eslint rules accordingly). I'll aim to submit a similar PR for the todos app shortly, to keep things in synch. Thanks!

@stubailo
Copy link
Contributor

@stubailo stubailo commented Apr 1, 2016

Can we add a short comment in the first code snippet that says why we're not using arrow functions?

Thanks!

Loading

@stubailo stubailo merged commit 4b649c6 into meteor:master Apr 1, 2016
2 checks passed
Loading
@hwillson
Copy link
Contributor Author

@hwillson hwillson commented Apr 1, 2016

Good idea - added via #326. Thanks!

Loading

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants