Skip to content

Linting and bugfixes #251

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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Open
burythehammer wants to merge 16 commits into
base: main
Choose a base branch
from
Open

Linting and bugfixes #251

burythehammer wants to merge 16 commits into from

Conversation

@burythehammer
Copy link
Collaborator

@burythehammer burythehammer commented Dec 19, 2025

Fixed:

  • Fixed nil pointer dereference when setting request headers before error check
  • Fixed race condition in rate limiter - mutex now released during sleep
  • Fixed context cancellation not being returned from rate limiter sleep
  • Fixed panic when database list page is empty
  • Fixed inconsistent receiver type in subscriptions List method
  • Fixed many other linting errors raised by new golangci-lint checks

Changed:

  • DeleteWithQuery now accepts a query parameter instead of always passing nil

Added:

  • Added golangci-lint to CI pipeline with comprehensive linting configuration

@burythehammer burythehammer self-assigned this Dec 19, 2025
@burythehammer burythehammer added bug Something isn't working github_actions Pull requests that update Github_actions code labels Dec 19, 2025
burythehammer and others added 7 commits December 19, 2025 17:46
@burythehammer
docs: changelog
ea983b8
@burythehammer
fix: golangci go version
6e4bcb4
@dependabot @burythehammer
chore(deps): bump golang.org/x/tools from 0.39.0 to 0.40.0
d63152d 
Bumps [golang.org/x/tools](https://github.com/golang/tools) from 0.39.0 to 0.40.0.
- [Release notes](https://github.com/golang/tools/releases)
- [Commits](golang/tools@v0.39.0...v0.40.0)

---
updated-dependencies:
- dependency-name: golang.org/x/tools
  dependency-version: 0.40.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@burythehammer
ci: moving linting before build to fail fast
c7e0b06
@burythehammer
ci: using v2 as latest pulls v1
d13671f
@burythehammer
ci: bump golang-lint-action version
9ecb840
@burythehammer
ci: more properly fixing version of golangci-lint
8bf693d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@burythehammer burythehammer

Labels

bug Something isn't working github_actions Pull requests that update Github_actions code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@burythehammer