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.

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

chore: update #27

Merged
merged 1 commit into from
Nov 26, 2024
Merged

chore: update #27

merged 1 commit into from
Nov 26, 2024

Conversation

dbarrosop
Copy link
Member

@dbarrosop dbarrosop commented Nov 26, 2024 •
edited by github-actions bot
Loading

PR Type

enhancement

Description

  • Updated golangci-lint to use buildGoModule instead of buildGo123Module.
  • Set golangci-lint version to "1.61.0".

Changes walkthrough 📝

Relevant files
Enhancement
go.nix
Update `golangci-lint` to use `buildGoModule` and set version

overlays/go.nix

  • Updated golangci-lint to use buildGoModule instead of
    buildGo123Module.
  • Set golangci-lint version to "1.61.0".
    		•  +1/-1     

    💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

    @github-actions GitHub Actions
    Copy link
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Code Consistency
    Ensure that the change from buildGo123Module to buildGoModule does not affect any other dependencies or modules that might rely on the previous configuration.

    @github-actions GitHub Actions
    Copy link
    Contributor

    PR Code Suggestions ✨

    @dbarrosop dbarrosop merged commit 7358bb7 into main Nov 26, 2024
    3 checks passed
    @dbarrosop dbarrosop deleted the upd branch November 26, 2024 08:55
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Reviewers

    @nunopato nunopato nunopato approved these changes

    Assignees
    No one assigned
    Labels
    Review effort [1-5]: 1
    Projects
    None yet
    Milestone
    No milestone
    Development

    Successfully merging this pull request may close these issues.
    2 participants
    @dbarrosop @nunopato