Add code linting to build #13

Merged

jacob1123 merged 57 commits from linting into master 2022-02-20 23:53:48 +01:00

Conversation 0 Commits 57 Files Changed 31 +600 -540

jacob1123 commented 2022-02-15 10:33:32 +01:00

Owner

Copy Link

No description provided.

jacob1123 added 2 commits 2022-02-15 10:33:32 +01:00

Add golang-ci to dev image 8f6974e151

Add vet fmt and linters to build 4bbbc0be13

jacob1123 added 1 commit 2022-02-15 12:52:31 +01:00

Ad config for golangci 835a15ec08

jacob1123 added 10 commits 2022-02-15 13:41:19 +01:00

Fix issues from golangci 38dfa540b4

Add explicit CI task to Taskfile d8e0f5a160

Add deps for dev-docker target aaf16dbe92

Add explicit CI task to Taskfile 584e7ef393

Enable all linters 71c54c9373

Ignore gin.Context for varnamelen 7a0c4a17a2

Wrap more errors 74a53954de

Move init of StaticFS and rename some vars 2f45c415e0

Disable deprecated linters 7b20bc9822

Improve error messages bb4548c50d

jacob1123 added 2 commits 2022-02-15 13:59:57 +01:00

Fix wrong executable name 737d5fb101

Minor fixes 72b5bdde4f

jacob1123 added 11 commits 2022-02-19 23:04:56 +01:00

Rename http package to server daadfd45bc

Rename parameter to other 0f2501dcbd

Remove context from YNABImport struct 649f937254

Disable wsl linter ... c5a0f49719

Seems to be excessive

Small refactorings 46b9b82f30

Disable forbidigo and nlreturn linters 052a2628ab

Enable formatting via gofumpt 1a19d3a197

Some linting fixes 02ba80a555

Disable linter testpackage 558fddc139

Return own error b0175542f1

Renames and dots in comments 1d2ae0e394

jacob1123 added 1 commit 2022-02-19 23:06:14 +01:00

Disable linting for prod build while open errors exist but enable for dev instead f0961ccc3c

jacob1123 added 1 commit 2022-02-19 23:13:57 +01:00

Fix missing value usage 77afe700ae

jacob1123 added 9 commits 2022-02-20 21:43:03 +01:00

Extract error consts c5be03ab6b

Run tests in parallel 9b92e2b551

Use short syntax for errcheck where possible 1a1971246d

Use camelCase for json identifiers 8035403416

Disable ifshort 787165b7f1

Remove cost and just use defaultCost b52ed21d1d

Fix capitalization of errors 96b514ccf8

Renames 36b2f12183

Use AbortWithStatusJSON instead of AbortWithError ca51ac5e27

jacob1123 added 12 commits 2022-02-20 22:33:31 +01:00

UNSURE explicitly init NullUUIDs 62085cb694

Reformat config.go 1dcd0d2f6d

Disable exhaustivstruct e39d1dc6e3

Revert "UNSURE explicitly init NullUUIDs" ... a0d89ee93a

This reverts commit 62085cb69439d36db301dc9097ae1f259fdd0d9e.

Use all gofiles as sources for go build 75a6ce1577

imports, comments and formatting 91b8cc06b2

Ignore forcetypeassert for token map d815e8c3cd

Run gci 260ac2d4ad

Comments 545f223a97

Minimal linting improvements c03d16878a

Disable gci 22ec0433bf

Improvements 2423bdd3ee

jacob1123 added 1 commit 2022-02-20 22:51:14 +01:00

Improve ynab import by extracting methods 6929c940c4

jacob1123 added 4 commits 2022-02-20 23:41:00 +01:00

Disable linter varnamelen e08a21b750

Fix multiple linter errors 3cb39d978a

Minor improvements 19d2ddbd65

Move comment 1bd38bb367

jacob1123 added 1 commit 2022-02-20 23:41:58 +01:00

Enable linting nfor CI cfda327a5d

jacob1123 added 2 commits 2022-02-20 23:52:03 +01:00

Remove commented code and TODOs 4688d2d94d

Get session secret from env instead of hardcoding 578e7d071c

jacob1123 merged commit a6eb2a2253 into master 2022-02-20 23:53:48 +01:00

jacob1123 referenced this issue from a commit 2022-02-20 23:53:48 +01:00

Merge pull request 'Add code linting to build' (#13) from linting into master

jacob1123 deleted branch linting 2022-02-23 21:54:05 +01:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Something is not working

duplicate

This issue or pull request already exists

enhancement

New feature

help wanted

Need some help

invalid

Something is wrong

question

More information is needed

wontfix

This won't be fixed

No Label

Milestone

No items

No Milestone

Assignees

Clear assignees

jacob1123

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: jacob1123/budgeteer#13

No description provided.