mirror of
https://codeberg.org/forgejo/forgejo.git
synced 2026-03-25 15:23:03 -04:00
757eb2f267
Some error types are used inconsistently or wrong: - `forgejo.org/modules/git.ErrNotExist` is meant to be a value error: <[modules/git/error.go#L23](https://codeberg.org/forgejo/forgejo/src/tag/v13.0.2/modules/git/error.go#L23)> - `forgejo.org/models/repo.ErrRepoNotExist` is meant to be a value error: <[models/repo/repo.go#L750](https://codeberg.org/forgejo/forgejo/src/tag/v13.0.2/models/repo/repo.go#L750)> - `errors.Is(logErr, &net.OpError{})` is always `false`: <[services/context/context_response.go#L188](https://codeberg.org/forgejo/forgejo/src/tag/v13.0.2/services/context/context_response.go#L188)> - `forgejo.org/models/issues.ErrIssueContentHistoryNotExist` is used inconsistently: <[models/issues/content_history.go#L211](https://codeberg.org/forgejo/forgejo/src/tag/v13.0.2/models/issues/content_history.go#L211)> Decided to use a value, since the structure is small and to be in line with the above errors. These issued where found with the [errortype](https://codeberg.org/fillmore-labs/errortype) linter and add this to Makefile as part of the linter suite. Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/9873 Reviewed-by: Gusted <gusted@noreply.codeberg.org> Co-authored-by: Oliver Eikemeier <eikemeier@fillmore-labs.com> Co-committed-by: Oliver Eikemeier <eikemeier@fillmore-labs.com> |
||
|---|---|---|
| .. | ||
| color.go | ||
| color_console.go | ||
| color_console_other.go | ||
| color_router.go | ||
| event_format.go | ||
| event_format_test.go | ||
| event_writer.go | ||
| event_writer_base.go | ||
| event_writer_buffer.go | ||
| event_writer_buffer_test.go | ||
| event_writer_conn.go | ||
| event_writer_conn_test.go | ||
| event_writer_console.go | ||
| event_writer_file.go | ||
| flags.go | ||
| flags_test.go | ||
| init.go | ||
| level.go | ||
| level_test.go | ||
| logger.go | ||
| logger_global.go | ||
| logger_impl.go | ||
| logger_impl_test.go | ||
| logger_test.go | ||
| manager.go | ||
| manager_test.go | ||
| misc.go | ||
| stack.go | ||