Skip to content

chore: modernize codebase#125

Merged
pavelnikolov merged 6 commits into
masterfrom
update-ci-workflows
May 27, 2026
Merged

chore: modernize codebase#125
pavelnikolov merged 6 commits into
masterfrom
update-ci-workflows

Conversation

@pavelnikolov
Copy link
Copy Markdown
Member

  • Update CI workflows, add lint checks.
  • Update Go version to v1.25
  • Fix lint errors

Copilot AI review requested due to automatic review settings May 27, 2026 10:42
@pavelnikolov pavelnikolov changed the title Update ci workflows chore: modernize codebase May 27, 2026
@github-advanced-security
Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

Copy link
Copy Markdown

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

Modernizes the repository’s tooling and code style by migrating CI to GitHub Actions, adding additional automated checks (lint/modernization), and updating Go/module settings and minor code/test cleanups.

Changes:

  • Replaced Travis CI with a GitHub Actions workflow that builds, tests with -race, runs golangci-lint, and enforces go fix/gofmt cleanliness.
  • Added a CodeQL workflow for Go security/code scanning.
  • Updated Go module version directive and applied small lint/style fixes across code, examples, and tests (including error string normalization).

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
README.md Updates build badge to GitHub Actions workflow.
go.mod Bumps the go directive to a newer Go version.
example_golang_lru_test.go Refactors the LRU cache adapter to avoid embedding and aligns method calls.
dataloader.go Normalizes panic error string formatting.
dataloader_test.go Updates tests to match new error strings and addresses minor lint/style issues.
cache.go Simplifies NOOP method bodies to empty blocks.
.github/workflows/go.yml Adds GitHub Actions CI with build/test/lint plus modernization checks.
.github/workflows/codeql-analysis.yml Introduces CodeQL scanning workflow for Go.

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

Comment thread dataloader_test.go Outdated
Comment thread .github/workflows/go.yml Outdated
@pavelnikolov pavelnikolov merged commit 4aab5c3 into master May 27, 2026
3 checks passed
@pavelnikolov pavelnikolov deleted the update-ci-workflows branch May 27, 2026 10:56
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.

3 participants