Feature/switch prefer-function-type eslint rule from warn to error (#3981)

* Switch prefer-function-type eslint rule from warn to error

* Update changelog

---------

Signed-off-by: Dominik Willner <th33xitus@gmail.com>

.eslintrc.json

@@ -142,7 +142,6 @@
 
         // The following rules are part of @typescript-eslint/stylistic-type-checked
         // and can be remove once solved
-        "@typescript-eslint/prefer-function-type": "warn",
         "@typescript-eslint/prefer-nullish-coalescing": "warn", // TODO: Requires strictNullChecks: true
         "@typescript-eslint/consistent-type-assertions": "warn",
         "@typescript-eslint/prefer-optional-chain": "warn",

CHANGELOG.md

@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Switched the `consistent-type-definitions` rule from `warn` to `error` in the `eslint` configuration
 - Switched the `no-empty-function` rule from `warn` to `error` in the `eslint` configuration
+- Switched the `prefer-function-type` rule from `warn` to `error` in the `eslint` configuration
 
 ### Fixed
 

apps/client/src/app/util/form.util.ts

@@ -8,7 +8,7 @@ export async function validateObjectForForm<T>({
   ignoreFields = [],
   object
 }: {
-  classDto: { new (): T };
+  classDto: new () => T;
   form: FormGroup;
   ignoreFields?: string[];
   object: T;