Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(lint): import golangci.yaml from GoToSocial and fix violations #3

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

milas
Copy link

@milas milas commented Jan 29, 2024

Notable changes here:

  • Small API breakage with Verifier::KeyId() -> Verifier::KeyID()
  • License is kept as BSD-3-Clause but with GoToSocial Authors added (cc @NyaaaWhatsUpDoc @daenney)
  • Exception made on var-naming rule for a bunch of constants that are otherwise unreadable without _ to separate things like a constant for the value sha512-256 (e.g. sha512256 instead of sha512_256 馃槢)

@milas milas changed the title [chore] Update readme + mod for fork, remove CI chore(lint): import golangci.yaml from GoToSocial and fix violations Jan 29, 2024
algorithms.go Show resolved Hide resolved
algorithms.go Show resolved Hide resolved
Notable changes here:
 * Small API breakage with `Verifier::KeyId()` -> `VerifierKeyID()`
   and `IsSupportedHttpSigAlgorithm()` -> `IsSupportedAlgorithm()`
 * License is kept as BSD-3-Clause but with GoToSocial Authors added
 * Exception made on `var-naming` rule for a bunch of constants that
   are otherwise unreadable without `_` to separate things like a
   constant for the value `sha512-256`

Signed-off-by: Milas Bowman <devnull@milas.dev>
algorithms.go Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants