NOAA-GSL / MATS

MATS is a quick & interactive way to view verification statistics
https://gsl.noaa.gov/mats/
6 stars 0 forks source link

Lint MATS app files #1047

Open mollybsmith-noaa opened 10 months ago

mollybsmith-noaa commented 10 months ago

Right now our linter is ignoring lots of problems in the MATS code. These problems need to be addressed, so that the ignores can be taken out of the linter config file. I've already done this for METexpress.

mollybsmith-noaa commented 9 months ago

I've been working on this, but I got bored so I switched branches to add visibility to the couchbase app. Will go back to linting afterwards.

mollybsmith-noaa commented 9 months ago

Made lots of progress on linting this week. I want to get at least the individual apps linted before the partial sums are ready. MATScommon is another beast.

mollybsmith-noaa commented 8 months ago

Still to be done: lint upper air, see if it is possible to remove all the query string replaces in the mysql apps and use string literals instead.

mollybsmith-noaa commented 8 months ago

Also add /global/ to mysql apps

mollybsmith-noaa commented 7 months ago

Upper air was linted as part of the cb-metar effort. I changed some function signatures in the common code, and it was easier to incorporate the changes into the upper air app if it looked like all the other apps.

github-actions[bot] commented 4 months ago

This issue is stale because it has been open 90 days with no activity.

mollybsmith-noaa commented 4 months ago

Not stale

github-actions[bot] commented 1 month ago

This issue is stale because it has been open 90 days with no activity.