1
0
mirror of https://github.com/Unleash/unleash.git synced 2024-12-22 19:07:54 +01:00
unleash.unleash/scripts
Gastón Fournier 7aed1e0d13
chore: gradually reduce null-check errors (#3094)
## About the changes

In order to move us towards enabling `strictNullChecks` we'd want to
have a way of gradually enabling this without having to fix all errors
at once, this will force us to start reducing the number of null check
issues.

This new workflow:
1. [Checks out the current branch and main into 2 different
folders](https://github.com/Unleash/unleash/pull/3094/files#diff-068f2ace1d1d2e773fb5e4240c83ccab251556fd5524fe13847122878e40da3bR15-R23)
2. Uses the **same** script `gradual-strict-null-checks.sh` (from the
current branch) [against each folder in
parallel](https://github.com/Unleash/unleash/pull/3094/files#diff-068f2ace1d1d2e773fb5e4240c83ccab251556fd5524fe13847122878e40da3bR34-R38)
to count the number of errors if `strictNullChecks` was enabled
3. If the number of potential errors in the current branch is higher
than the number of potential errors in main [it
fails](https://github.com/Unleash/unleash/pull/3094/files#diff-068f2ace1d1d2e773fb5e4240c83ccab251556fd5524fe13847122878e40da3bR41-R46)

As an example, a [new issue was introduced in this
PR](753f57223c)
(and then
[reverted](e4deb62965)),
so we can test the build failure:

https://github.com/Unleash/unleash/actions/runs/4163632636/jobs/7204268519#step:5:10


## Discussion points
This could be a non-mandatory check, just advising, or even adding a
comment in the PR. It might be good to start with a non-strict check,
but at the same time we can decide to make it non-strict if a problem
appears

In some situations, an additional null check error might require us to
fix a bunch of them, increasing the time to deliver. In these cases we
can suppress an individual line with `// @ts-ignore: Object is possibly
'null'.` although might defeat the purpose of this workflow
2023-02-14 15:52:21 +01:00
..
check-release.sh fix: add frontend eslint check (#2562) 2023-01-02 10:46:32 +01:00
docker-compose.yml chore(deps): update dpage/pgadmin4 docker tag to v6.19 (#2910) 2023-01-17 18:35:06 +00:00
docker-postgres.sh Chore: rework docker-postgres.sh to be cross-platform (#1037) 2021-10-23 09:12:34 +02:00
fix-migrations-version-1.sql Added doc to fix migrations table 2020-02-20 08:30:44 +01:00
gradual-strict-null-checks.sh chore: gradually reduce null-check errors (#3094) 2023-02-14 15:52:21 +01:00
husky-install.js
jest-setup.js
npm-tag.js
resetdatabase.sql Add quick reset of database script 2021-03-11 11:18:06 +01:00
wait-for fix: move docker-compose to this repo (#2666) 2022-12-12 20:40:18 +01:00