Declare `Raphael` as a global for JSHint
ClosedPublic
Actions

Authored by joshuaspence on Dec 30 2014, 10:29 AM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

Restricted Diffusion Commit
rPdb563426156a: Declare `Raphael` as a global for JSHint

Summary

Let JSHint know that Raphael is a global that can be used anywhere. This is not technically correct, but it silences a few JSHint warnings. See http://jshint.com/docs/ for more information.

Test Plan

arc lint -- webroot/rsrc/js/application/maniphest/behavior-line-chart.js

Diff Detail

Repository

rP Phabricator

Branch

master

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 3364
Build 3371: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence updated this revision to Diff 26562.Dec 30 2014, 10:29 AM

joshuaspence retitled this revision from to Declare `Raphael` as a global for JSHint.

joshuaspence updated this object.

joshuaspence edited the test plan for this revision. (Show Details)

joshuaspence added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptDec 30 2014, 10:29 AM

Herald added subscribers: epriestley, Korvin. · View Herald Transcript

epriestley accepted this revision.Dec 30 2014, 10:58 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Dec 30 2014, 10:58 AM

Closed by commit rPdb563426156a: Declare `Raphael` as a global for JSHint (authored by joshuaspence, committed by epriestley). · Explain WhyDec 30 2014, 10:58 AM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in D11071: Define a seperate JSHint configuration for NodeJS files.Dec 30 2014, 11:01 AM

Revision Contents
Changeset List

Path

Size

support/

lint/

jshintrc

1 line

Diff 26562

View Options

Declare `Raphael` as a global for JSHintClosedPublicActions