Skip to content

chore: bump tooling/deps and refactor scheduler counter to atomic.Uint64#34

Merged
hyp3rd merged 1 commit intomainfrom
feat/scheduler
Apr 4, 2026
Merged

chore: bump tooling/deps and refactor scheduler counter to atomic.Uint64#34
hyp3rd merged 1 commit intomainfrom
feat/scheduler

Conversation

@hyp3rd
Copy link
Copy Markdown
Owner

@hyp3rd hyp3rd commented Apr 4, 2026

  • Update golangci-lint from v2.11.3 → v2.11.4 across pre-commit hook, project settings, and Makefile
  • Update buf from v1.66.1 → v1.67.0
  • Bump github.com/hyp3rd/ewrap v1.3.7 → v1.3.9
  • Bump modernc.org/sqlite v1.46.1 → v1.48.1
  • Replace plain uint64 counter field in Scheduler with atomic.Uint64, eliminating the need for the atomic.AddUint64 package-level call in favour of the safer, self-contained s.counter.Add(1) method

- Update golangci-lint from v2.11.3 → v2.11.4 across pre-commit hook,
  project settings, and Makefile
- Update buf from v1.66.1 → v1.67.0
- Bump github.com/hyp3rd/ewrap v1.3.7 → v1.3.9
- Bump modernc.org/sqlite v1.46.1 → v1.48.1
- Replace plain uint64 counter field in Scheduler with atomic.Uint64,
  eliminating the need for the atomic.AddUint64 package-level call in
  favour of the safer, self-contained s.counter.Add(1) method
Copilot AI review requested due to automatic review settings April 4, 2026 09:40
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates core tooling/dependency versions and refactors the scheduler’s internal job ID counter to use sync/atomic.Uint64 for safer, self-contained atomic increments.

Changes:

  • Bumped golangci-lint and buf versions across project settings, Makefile, and pre-commit hook.
  • Updated Go module dependencies (github.com/hyp3rd/ewrap, modernc.org/sqlite) and corresponding go.sum entries.
  • Refactored Scheduler.counter from uint64 + atomic.AddUint64 to atomic.Uint64 + s.counter.Add(1).

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pkg/scheduler/scheduler.go Switches scheduler ID counter to atomic.Uint64 and uses the typed Add method.
Makefile Updates pinned golangci-lint and buf versions used by tooling targets.
go.mod Bumps ewrap and modernc.org/sqlite module versions.
go.sum Updates checksums to match the new module versions.
.project-settings.env Updates shared tooling version pins used by scripts/hooks.
.pre-commit/golangci-lint-hook Updates default golangci-lint version used by the hook.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@hyp3rd hyp3rd merged commit 54ff37a into main Apr 4, 2026
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants