Remove old TODOs

database/database.go

@@ -11,8 +11,6 @@ import (
 	errorutil "goyave.dev/goyave/v5/util/errors"
 )
 
-// TODO document there is no initializer and registered models anymore, the view interface has been removed, no auto migrations
-
 // New create a new connection pool using the settings defined in the given configuration.
 //
 // In order to use a specific driver / dialect ("mysql", "sqlite3", ...), you must not

database/paginator.go

@@ -8,8 +8,6 @@ import (
 	"goyave.dev/goyave/v5/util/errors"
 )
 
-// TODO find a smooth way to convert a paginator's records to DTO (typeutil.Convert simply?)
-
 // Paginator structure containing pagination information and result records.
 type Paginator[T any] struct {
 	DB *gorm.DB `json:"-"`

util/errors/error.go

@@ -232,7 +232,3 @@ func (r Reason) Error() string {
 func (r Reason) MarshalJSON() ([]byte, error) {
 	return json.Marshal(r.reason)
 }
-
-// TODO document exit codes are removed
-// Exit code doesn't really bring value, the error message in the logs is more important
-// The server now returns *errors.Error

validation/size.go

@@ -14,7 +14,7 @@ func validateSize(value any, v func(size int) bool) bool {
 	switch getFieldType(val) {
 	case FieldTypeString:
 		return v(uniseg.GraphemeClusterCount(value.(string)))
-	case FieldTypeArray, FieldTypeObject: // TODO document it also works for objects (number of keys)
+	case FieldTypeArray, FieldTypeObject:
 		return v(val.Len())
 	case FieldTypeFile:
 		files, _ := value.([]fsutil.File)

validation/validator.go

@@ -366,7 +366,6 @@ func (v *validator) validateField(fieldName string, field *Field, walkData any,
 				Name:      c.Name,
 				path:      errorPath,
 				Invalid:   !valid,
-				// TODO add validation errors Merge
 			}
 			validator.init(v.options) // TODO document a Rules or RuleSet is not meant to be re-used or used concurrently
 			ok := validator.Validate(ctx)