Create tool for linting Arduino projects #1

per1234 · 2020-09-22T04:56:38Z

No description provided.

project/project.go

project/platform/platform.go

project/packageindex/packageindex.go

check/checklevel/checklevel.go

project/sketch/sketch.go

check/checkfunctions/sketch.go

configuration/checkmode/checkmode.go

configuration/configuration.go

project/library/library.go

result/result.go

check/check.go

silvanocerza · 2020-10-27T17:18:50Z

I watched the whole PR and left lots of comments but I must have missed something for sure.
I've not commented some indentation issues but would love if you could fix those too.
Also I've noticed you call String() lots of times when printing or logging, I'm not completely sure but it shouldn't be necessary most of the times since you're already using the %s verb for formatting.

silvanocerza

🚀

per1234 · 2020-10-29T18:32:37Z

I watched the whole PR and left lots of comments

Thanks so much. Great suggestions!

I've not commented some indentation issues but would love if you could fix those too.

I have run task go:format on the project without any changes. However, it's possible that there are style choices that are not enforced by go fmt. I'm very much receptive to any suggestions for formatting improvements. If you could provide me with an example, I can then fix that issue and also do a survey of the code to apply the same sort of improvement comprehensively.

Also I've noticed you call String() lots of times when printing or logging, I'm not completely sure but it shouldn't be necessary most of the times since you're already using the %s verb for formatting.

There was definitely a lot of unnecessary use of String(). I believe I have removed all of them in 73d8eab and what remains is all necessary. I guess it was propagated from some early use of print() and then I just never realized it wasn't needed anymore after updating the output code.

- Put type in dedicated package to create a namespace for it - Name custom type "Type" (the package name provides the identification)

Checks should always be explicitly configured to run or not run in any check mode.

I made a dedicated function for this purpose, then forgot to use it!

…ry package

…package

This information is not normally of interest.

When a report file path is specified by the user, a JSON formatted report of the checks will be written to that path.

Since the variables already contain the value that needs to be returned, the if statement only adds unnecessary verbosity.

This would indicate a new project type was added to the tool without updating the subproject discovery code accordingly.

Previously, the more general project filter type was used, but the specific project type that was detected will be more useful information.

The term "project type" is used both to refer to the type of the project filter, which may be "all", and to the specific project type (e.g., sketch, library, platform). This can result in some ambiguity in the code. The use of "filter" in variable names that are used exclusively for project type filters may make the code easier to follow.

Consolidate the redundant logging code in these functions.

Previously, there was no provision for multiple report instances. Although the current tool design has no need for multiple instances, the code now has the versatility to allow for report demands that might arise in the future.

The change in approach in the result package resulted in some of the use of the term "report" in that package becoming less intuitive, since reports are only one use of the results.

per1234 force-pushed the development branch 5 times, most recently from 6434353 to 8b5a200 Compare September 22, 2020 23:18

per1234 force-pushed the development branch from 8b5a200 to 7ef7cf2 Compare October 3, 2020 15:17

per1234 force-pushed the development branch from 7ef7cf2 to d84106a Compare October 22, 2020 13:07