summaryrefslogtreecommitdiff
path: root/.gitlab-ci/run-check-todos.sh
Commit message (Collapse)AuthorAgeFilesLines
* Move common code for searching common ancestor in a dedicated scriptFrederic Martinsons2021-01-111-24/+1
| | | | Signed-off-by: Frederic Martinsons <frederic.martinsons@sigfox.com>
* ci: Limit depth of fetch for initial code checksPhilip Withnall2020-07-251-1/+14
| | | | | | | | | | | | Rather than fetching the entire repository history, limit the fetch depth to commits in the last 4 weeks. If a branch was branched longer than 4 weeks ago, it seems reasonable to require it to be rebased before it can be reviewed. At the current rate of development, this reduces the bandwidth needed for git pulls in the initial code checks from 73MB to 9MB. Signed-off-by: Philip Withnall <withnall@endlessm.com>
* tree: Fix various typos and outdated terminologyPhilip Withnall2020-06-121-2/+3
| | | | | | | | | | | | | | | | | | | | | | | This was mostly machine generated with the following command: ``` codespell \ --builtin clear,rare,usage \ --skip './po/*' --skip './.git/*' --skip './NEWS*' \ --write-changes . ``` using the latest git version of `codespell` as per [these instructions](https://github.com/codespell-project/codespell#user-content-updating). Then I manually checked each change using `git add -p`, made a few manual fixups and dropped a load of incorrect changes. There are still some outdated or loaded terms used in GLib, mostly to do with git branch terminology. They will need to be changed later as part of a wider migration of git terminology. If I’ve missed anything, please file an issue! Signed-off-by: Philip Withnall <withnall@endlessm.com>
* ci: Ensure run-check-todos.sh exits on the first error it encountersPhilip Withnall2020-02-271-1/+1
| | | | Signed-off-by: Philip Withnall <withnall@endlessm.com>
* ci: Fix various shellcheck warnings in the shell scriptsPhilip Withnall2020-02-271-1/+1
| | | | | | | I haven’t tested any of them. This is entirely mechanical. I used shellcheck 0.7.0 with default options. Signed-off-by: Philip Withnall <withnall@endlessm.com>
* ci: Add checks for banned keywords in merge requestsPhilip Withnall2020-01-211-0/+16
Using the same approach as we have for code style checks (the `style-check-diff` CI job), check the diff for any banned keywords like ‘TODO’, and also check the commit messages. The keyword ‘TODO’ is often used by developers to indicate a part of a commit which needs further work, and hence which shouldn’t yet be merged. Signed-off-by: Philip Withnall <withnall@endlessm.com> Fixes: #1551