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

Some more cleanups for heatmap #29

Merged
merged 1 commit into from Aug 23, 2017
Merged

Some more cleanups for heatmap #29

merged 1 commit into from Aug 23, 2017

Conversation

@c-w
Copy link
Contributor

@c-w c-w commented Aug 23, 2017

No functional change, just some refactoring:

  • Convert let to const where possible
  • Extract helper methods
  • Make constant naming consistent
  • Prefer early-return over nesting
No functional change, just some refactoring:

- Convert `let` to `const` where possible
- Extract helper methods
- Make constant naming consistent
- Prefer early-return over nesting
@c-w c-w requested a review from Smarker Aug 23, 2017
Copy link
Contributor

@Smarker Smarker left a comment

LGTM

@c-w c-w merged commit 997b797 into master Aug 23, 2017
@c-w c-w deleted the cleanup-heatmap branch Aug 23, 2017
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