overleaf/server-ce/init_scripts
Antoine Clausse fe03d8d7fd Add shellcheck to web, server-pro, server-ce (#23823)
* Update shellcheck files to include non ".sh" files having the sh shebang

* Add shellcheck to root Makefile

* `make shellcheck_fix`

* Add shellcheck to server-ce and server-pro

* Exclude SC1091 (Not following) in SP/CE

* Fix errors in SP

* Fix errors in CE

* Update root shellcheck to ignore all failing rules. We can later remove rules one by one.

* Add shellcheck to web

* Add shellcheck step for server-ce and server-pro in cloudbuild.yaml

* Revert "Add shellcheck to root Makefile"

This reverts commit e0fa56f2

* Revert "`make shellcheck_fix`"

This reverts commit eb179245c109a9e742a7fdeeb75a4bdd03963587.

* `make shellcheck_fix` in server-ce

* Fix: Use $(...) notation instead of legacy backticked

```
In init_scripts/100_set_docker_host_ipaddress.sh line 5:
echo "`route -n | awk '/UG[ \t]/{print $2}'` dockerhost" >> /etc/hosts
      ^-- SC2006 (style): Use $(...) notation instead of legacy backticks `...`.

Did you mean:
echo "$(route -n | awk '/UG[ \t]/{print $2}') dockerhost" >> /etc/hosts

For more information:
  https://www.shellcheck.net/wiki/SC2006 -- Use $(...) notation instead of le...
```

* `make shellcheck_fix` in web

* Manual corrections on autofix

* Update SHELLCHECK_FILES to Perl regex so it matches at start of file

GitOrigin-RevId: 815d53f319a3792efa22703beb75570de5977450
2025-02-26 09:05:02 +00:00
..
000_check_for_old_bind_mounts_5.sh
000_check_for_old_env_vars_5.sh
100_generate_secrets.sh
100_make_overleaf_data_dirs.sh
100_restore_site_status.sh
100_set_docker_host_ipaddress.sh Add shellcheck to web, server-pro, server-ce (#23823) 2025-02-26 09:05:02 +00:00
200_nginx_config_template.sh Add shellcheck to web, server-pro, server-ce (#23823) 2025-02-26 09:05:02 +00:00
300_delete_old_logs.sh
500_check_db_access.sh Merge pull request #22580 from overleaf/ar-migrate-server-ce-scripts 2025-01-09 09:05:54 +00:00
900_run_web_migrations.sh
910_check_texlive_images Merge pull request #22580 from overleaf/ar-migrate-server-ce-scripts 2025-01-09 09:05:54 +00:00
910_initiate_doc_version_recovery