Commit graph

7 commits

Author SHA1 Message Date
Feignint
8261493a4c Add extra_bash_files array to scripts/verify-bash.sh
contrib/linux/bash-completion/dosbox should be
linted by shellcheck, but lacks shebang.

This commit 'manually' adds file to list

Currently it siliently skips missing files listed
in the extra_bash_files array, It may be prudent
to emitt a warning in future.
2021-08-14 08:30:02 -07:00
Patryk Obara
0a36c37328 Adjust copyright statements for 2021
- Update DOSBox Staging Team copyright span to year 2021 (DOSBox Team
  copyrights left untouched).
- Add DOSBox Staging Team copyright line to files we extensively
  modified throughout 2020 (we add our copyright lines when number of
  non-trivially modified lines gets close to 50%).
- Change "dosbox-staging team" to "DOSBox Staging Team", in line with
  updated branding.
- Change MAME license identifiers to SPDX license identifiers.
- Cleanup some excessive whitespace around copyright statements.
- Do few adjustments to include order.
- Add few missing copyright identifiers (GPL-2.0-or-later,
  LGPL-2.1-or-later, BSD-3-Clause) as appropriate for specific file.
- Move SPDX identifiers to the top in scripts for consistency with other
  source files.
- Remove old copy of GPL2 license in italian from old translations.
2021-01-01 00:12:05 +01:00
Feignint
204e75b97c Add GitHub Annotations feature to verify-bash.sh
This adds Annotations to the Job run's Summary and
in-line Annotations in the "Files Changed" tab

When env. var. GITHUB_ACTOR is present:

SHELLCHECK(1)'s json1 output format is used, JQ(1)
then outputs the GitHub API command:

    ::error file=<filename>,line=<line no>::<msg>

A simple GREP(1) is used to flip the return code,
i.e. "0" if no errors found and "123" when they
are. "123" is to keep in sync with the scripts
"normal", from XARGS(1) return code.

All of SHELLCHECK(1)'s severity levels:

    error, warning, info, style

Are reported as "error" for GitHub's API,
regardless of the severity (.comments[].level)
2020-12-18 14:22:53 +01:00
Patryk Obara
3be0409d25 Update copyright line in several scripts
Use the exact formatting suggested in COPYING file.

This change is basically a pretext to trigger a new clean build in
order to check if Coverity is capable of accepting new builds for
analysis finally.
2020-01-03 17:15:25 +01:00
Patryk Obara
5b99e786f0 Colorize linter output 2019-11-16 02:27:24 +01:00
Patryk Obara
2b386fbf52 List files before running linters
Makes it more readable for human consumption. Also, redirect initial
output to stderr to fix e.g. output to json file.
2019-11-16 02:27:24 +01:00
Patryk Obara
2bfae63f1f Run fast script linters before heavier analysis
Includes two small scripts: verify-bash.sh for running shellcheck, and
verify-python.sh for running pylint.

.pylint rc files is a default configuration file generated by
pylint 2.3.1, with one change (min-similarity-lines changed
from 4 to 10).
2019-11-14 20:33:36 +01:00