Skip to content

Commit

Permalink
Remove var in Libraries/vendor/core/merge.js (#22108)
Browse files Browse the repository at this point in the history
Summary:
Replaces the keywords var with const in Libraries/merge.js

If this PR fixes an issue, type "Fixes #issueNumber" to automatically close the issue when the PR is merged.

Pull requests that expand test coverage are more likely to get reviewed. Add a test case whenever possible!

- [x]  Check npm run flow
- [x] Check npm run flow-check-ios
- [x] Check npm run flow-check-android

[GENERAL] [ENHANCEMENT] [Libraries/core] - remove var
Pull Request resolved: #22108

Differential Revision: D12919282

Pulled By: TheSavior

fbshipit-source-id: e3c8569c47c95ad3d1a061917c90185c2694c78f
  • Loading branch information
yushimatenjin authored and facebook-github-bot committed Nov 4, 2018
1 parent e835c6d commit 3f069f3
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions Libraries/vendor/core/merge.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

"use strict";

var mergeInto = require('mergeInto');
const mergeInto = require('mergeInto');

/**
* Shallow merges two structures into a return value, without mutating either.
Expand All @@ -16,8 +16,8 @@ var mergeInto = require('mergeInto');
* @param {?object} two Optional object with properties to merge from.
* @return {object} The shallow extension of one by two.
*/
var merge = function(one, two) {
var result = {};
const merge = function(one, two) {
const result = {};
mergeInto(result, one);
mergeInto(result, two);
return result;
Expand Down

0 comments on commit 3f069f3

Please sign in to comment.