Enable no-dot-import lint check
All checks were successful
CI / release-test (push) Successful in 1m36s
All checks were successful
CI / release-test (push) Successful in 1m36s
This commit is contained in:
parent
dbde3c8724
commit
912210cc90
@ -58,9 +58,10 @@ linters:
|
||||
checks:
|
||||
- all
|
||||
- -ST1000 # Re-enable this once we have docstrings
|
||||
- -ST1001 # Dot imports are good sometimes (e.g., in test packages)
|
||||
- -ST1003 # I like snake_case
|
||||
- -ST1013 # HTTP status codes are shorter and more readable than names
|
||||
dot-import-whitelist:
|
||||
- "git.offline-twitter.com/offline-labs/gas-stack/pkg/flowutils"
|
||||
exclusions:
|
||||
generated: lax # Don't lint generated files
|
||||
paths:
|
||||
|
||||
@ -1,8 +1,8 @@
|
||||
TODO: auto-timestamps
|
||||
- SaveXyz should set created_at and updated_at; shouldn't touch is_deleted or deleted_at
|
||||
- if soft delete is enabled, DeleteXyz should do update (not delete) and set is_deleted and deleted_at
|
||||
- ...and DeleteXyz should have pointer receiver for soft-delete
|
||||
- SaveXyz shouldn't set created_at in the do-update branch
|
||||
- DeleteXyz should have pointer receiver
|
||||
- GetXyzByID should include `ErrItemIsDeleted` if item is soft-deleted
|
||||
|
||||
TODO: primary-key
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user