As of Drupal 8 all JS needs to follow our JavaScript coding standards. For this we use ESLint to check most of our standards.
Here is the summary of running ESLint on this module JS:
XXX problems.

Attached is the full list of errors found by ESLint and the patch that fixes them.

Comments

Purnachandra Rao V created an issue. See original summary.

krknth’s picture

Assigned: Purnachandra Rao V » krknth

@Purnachandra Rao V : Thanks, checking your patch.

Purnachandra Rao V’s picture

Status: Active » Needs review
Purnachandra Rao V’s picture

Please check this patch and confirm.

krknth’s picture

Status: Needs review » Reviewed & tested by the community
heykarthikwithu’s picture

Assigned: krknth » Unassigned
Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs reroll

patch does not apply to latest code, needs a re roll.

Purnachandra Rao V’s picture

StatusFileSize
new4 KB

Latest file some one added code.
Please find updated patch.

Purnachandra Rao V’s picture

Status: Needs work » Needs review
heykarthikwithu’s picture

Version: 8.x-1.0-alpha1 » 8.x-1.x-dev
Assigned: Unassigned » heykarthikwithu
Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs reroll

Looks fine.

heykarthikwithu’s picture

Assigned: heykarthikwithu » Unassigned
Status: Reviewed & tested by the community » Fixed

Committed, thanks.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.