search

brackets-archive / bracketsIssues

Archive of issues in brackets.
0 stars 0 forks source link

[CLOSED] Explicitly enable JSLint before running JSLint unit tests #4061

Open core-ai-bot opened 3 years ago

core-ai-bot commented 3 years ago

Issue by iwehrman Monday Jul 08, 2013 at 18:08 GMT Originally opened as https://github.com/adobe/brackets/pull/4384

Edge Code disables JSLint by default, so the new JSLint unit tests fail because JSLint is not enabled. This pull updates the unit tests to explicitly enable JSLint in the test window before running each test.

iwehrman included the following code: https://github.com/adobe/brackets/pull/4384/commits

core-ai-bot commented 3 years ago

Comment by RaymondLim Tuesday Jul 09, 2013 at 22:00 GMT

Looks good except that this can create an issue on Mac menubar since it is shared by all document windows. If you disable JSLint in View menu before running JSLint unit tests, you will see it enabled under View menu even though it is not really enabled in the primary document window. But this is a known issue with toggling menu settings in the new Brackets window and not that you introduced.

So merging now.