Review and refine linting includes/excludes.

Description

In looking into FLUID-6327, I started to suspect that the includes used to determine which files were linted were not quite complete. For example, there are a range of markdown files in the demos and examples directories that are not caught by the current linting pattern.

I reviewed in depth to get an idea of what needed to be done. Here's the current output of the linting tasks:

I compared that to a raw head count for all file types using the following command:

The results are as follows:

File Type

Files

md

13

js

247

json

88

json5

0

Looks like at 2 JSON files and 9 markdown files are not included in our linting. The javascript discrepancy is probably down to the "lib" files, but I will review that as well.

Environment

None

Activity

Show:

Tony Atkins [RtF] August 22, 2018 at 5:32 PM

Here's an updated command that excludes nearly all of the content that shouldn't be linted. Notably the fluid 1.5 js file is still included, but the totals are otherwise a good match for the linting output.

Tony Atkins [RtF] August 22, 2018 at 12:23 PM

Fixed

Details

Assignee

Reporter

Priority

Created August 22, 2018 at 10:25 AM
Updated July 22, 2024 at 2:35 PM
Resolved August 23, 2018 at 2:56 PM