BranchMetrics / web-branch-deep-linking-attribution

The Branch Web SDK for deep linking and attribution. Once initialized, the Branch Web SDK allows you to create and share links with a banner (web only), over SMS, or your own methods by generating deep links. It also offers event tracking, access to referrals, and management of credits.
https://help.branch.io/developers-hub/docs/web-sdk-overview
MIT License
287 stars 101 forks source link

Perf() : replace jscs with eslint #933

Open JagadeeshKaricherla-branch opened 10 months ago

JagadeeshKaricherla-branch commented 10 months ago

Pull Request Template

Description

Replace linting tool from jscs ( package deprecated ) to eslint

Fixes # (issue)

Type of change : performance

Please delete options that are not relevant.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

JS Budget Check

Please mention the size in kb before abd after this PR

Files Before After
dist/build.js.
dist/build.min.js

Checklist:

Mentions:

List the person or team responsible for reviewing proposed changes.

cc @BranchMetrics/saas-sdk-devs for visibility.

github-actions[bot] commented 10 months ago

Test Results

112 tests   105 :heavy_check_mark:  56s :stopwatch:     1 suites      7 :zzz:     1 files        0 :x:

Results for commit 9a0b7705.

:recycle: This comment has been updated with latest results.

codecov[bot] commented 10 months ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ce035be). Click here to learn what that means. The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #933   +/-   ##
=========================================
  Coverage          ?   24.60%           
=========================================
  Files             ?       15           
  Lines             ?     2321           
  Branches          ?        0           
=========================================
  Hits              ?      571           
  Misses            ?     1750           
  Partials          ?        0           
gdeluna-branch commented 10 months ago

Most of these changes are just from executing the eslint tool right?