light clean up #4

Merged

karasz merged 5 commits from pr-amery-cleanup into master 1 year ago

amery commented 1 year ago

Owner

add key words to vscode's dictionary (cspell)
simplify/standarize function names to avoid repeating the package name
refactor error handling so errors can be properly handled downstream
cleanup parser

amery added 4 commits 1 year ago

47d0d5d0a4 fix go.mod version

having 1.21.1 there makes go fmt require 1.23 to check the files

Signed-off-by: Alejandro Mery <amery@jpi.io>

5941fb013c vscode: add hasher, htpasswd and Passwds to the dictionary

Signed-off-by: Alejandro Mery <amery@jpi.io>

7b8b0e6700 htpasswd: standarize function names

Signed-off-by: Alejandro Mery <amery@jpi.io>

02235c36bc htpasswd: clean up errors

Signed-off-by: Alejandro Mery <amery@jpi.io>

amery added 1 commit 1 year ago

0fb985026f htpasswd: clean up parser

Signed-off-by: Alejandro Mery <amery@jpi.io>

amery requested review from karasz 1 year ago

karasz approved these changes 1 year ago

karasz left a comment

Owner

LGTM

❤️ 1

karasz merged commit 4395a40036 into master 1 year ago

karasz deleted branch pr-amery-cleanup 1 year ago

karasz referenced this issue from a commit 1 year ago

Merge pull request 'light clean up' (#4)

Reviewers

karasz approved these changes 1 year ago

The pull request has been merged as 4395a40036.

Sign in to join this conversation.

No Label

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.