mirror of
https://github.com/ddnet/ddnet.git
synced 2024-11-10 01:58:19 +00:00
fba7211740
5905: Update Tutorial map by unique_2 r=heinrich5991 a=def- > We made a small update of the tutorial map because of the new freeze > graphics. We also made the teleport tiles the airjump hints > consistent. Idk how updating the map works on your end, ideally > everyone keeps their time because the changes don't influence anyone's > times. <!-- What is the motivation for the changes of this pull request? --> <!-- Note that builds and other checks will be run for your change. Don't feel intimidated by failures in some of the checks. If you can't resolve them yourself, experienced devs can also resolve them before merging your pull request. --> ## Checklist - [ ] Tested the change ingame - [ ] Provided screenshots if it is a visual change - [ ] Tested in combination with possibly related configuration options - [ ] Written a unit test (especially base/) or added coverage to integration test - [ ] Considered possible null pointers and out of bounds array indexing - [ ] Changed no physics that affect existing maps - [ ] Tested the change with [ASan+UBSan or valgrind's memcheck](https://github.com/ddnet/ddnet/#using-addresssanitizer--undefinedbehavioursanitizer-or-valgrinds-memcheck) (optional) Co-authored-by: def <dennis@felsin9.de>
83 lines
3.3 KiB
YAML
83 lines
3.3 KiB
YAML
name: Check style
|
|
|
|
on:
|
|
push:
|
|
branches-ignore:
|
|
- master
|
|
- staging.tmp
|
|
- trying.tmp
|
|
- staging-squash-merge.tmp
|
|
pull_request:
|
|
|
|
jobs:
|
|
check-style:
|
|
runs-on: ubuntu-20.04
|
|
steps:
|
|
- uses: actions/checkout@v2
|
|
with:
|
|
submodules: true
|
|
- name: Prepare
|
|
run: |
|
|
sudo apt-get update -y
|
|
sudo apt-get install clang-format imagemagick ddnet-tools shellcheck pkg-config cmake ninja-build libfreetype6-dev libnotify-dev libsdl2-dev libsqlite3-dev libavcodec-dev libavformat-dev libavutil-dev libswresample-dev libswscale-dev libx264-dev python3-clang libvulkan-dev glslang-tools spirv-tools rustc cargo -y
|
|
pip3 install pylint
|
|
git clone https://gitlab.com/Patiga/twmap
|
|
cd twmap/twmap-tools
|
|
cargo install --path=.
|
|
cd ../..
|
|
rm -rf twmap
|
|
mkdir release
|
|
cd release
|
|
cmake -G Ninja -DCMAKE_BUILD_TYPE=Release -DDOWNLOAD_GTEST=OFF -DCMAKE_RUNTIME_OUTPUT_DIRECTORY_RELEASE=. ..
|
|
cmake --build . --config Release --target dilate
|
|
- name: Check clang-format
|
|
run: clang-format -version
|
|
- name: Check fix_style
|
|
run: scripts/fix_style.py --dry-run
|
|
- name: Check header guards
|
|
run: scripts/check_header_guards.py
|
|
- name: Check languages
|
|
run: scripts/languages/update_all.py
|
|
- name: Check dilated images
|
|
run: scripts/check_dilate.sh release data
|
|
# TODO: Enable on release branches
|
|
#- name: Out-of-date translations
|
|
# run: |
|
|
# cp -a data/languages data/languages.orig
|
|
# scripts/languages/update_all.py
|
|
# diff_lang=$(diff data/languages.orig data/languages)
|
|
# if [ -n "$diff_lang" ]; then
|
|
# echo "Update translations:\n$diff_lang"
|
|
# exit 1
|
|
# fi
|
|
- name: Shellcheck
|
|
run: find . -type f -name '*.sh' -print0 | xargs -0 shellcheck
|
|
- name: Check log error case
|
|
run: |
|
|
if grep -Eqr '(msg|Print).*\(.*"[Ee]rror:' src/;
|
|
then
|
|
echo "Expected log errors to be in this format 'ERROR: error message'"
|
|
echo "Found these non uppercased log errors:"
|
|
grep -Er '(msg|Print).*\(.*"[Ee]rror:' src/
|
|
exit 1
|
|
fi
|
|
- name: Pylint
|
|
run: |
|
|
pylint --version
|
|
find . -type f -name "*.py" -not -path './ddnet-libs/*' -not -path './googletest-src/*' -print0 | xargs -0 pylint
|
|
- name: Unused headers
|
|
run: |
|
|
find src -name '*.h' | while read -r i; do grep -r -q "$(basename "$i")" || (echo "Header file $i is unused" && exit 1); done
|
|
- name: Check maps
|
|
run: |
|
|
findings=$(find data -type f -name '*.map' -print0 | xargs -0 ~/.cargo/bin/twmap-check-ddnet 2>&1 | \
|
|
grep -v -E "\"data/maps/(ctf|dm).*\.map\": Time cps are missing on the map" | \
|
|
grep -v -E "\"data/themes/.*\.map\": Time cps are missing on the map" | \
|
|
grep -v -E "\"data/maps/ctf4\.map\": Map - Image error at index 3 : 'jungle_doodads_old' is not a valid external image in version DDNet06" | \
|
|
grep -v -E "\"data/themes/autumn_night\.map\": Map - Image error at index 3 : 'generic_lamps' is not a valid external image in version DDNet06" | \
|
|
grep -v -E "\"data/maps7?/Tutorial\.map\": Map setting - Unusual: 'sv_show_others_default'" || true)
|
|
if [ -n "$findings" ]; then
|
|
echo "$findings"
|
|
exit 1
|
|
fi
|