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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update dependencies #607

Merged
merged 3 commits into from
Feb 3, 2025
Merged

chore: update dependencies #607

merged 3 commits into from
Feb 3, 2025

Conversation

dbarrosop
Copy link
Member

@dbarrosop dbarrosop commented Feb 3, 2025

PR Type

Enhancement


Description

  • Update Go version and toolchain

  • Upgrade multiple dependencies to latest versions

  • Update PostgreSQL client version in flake.nix

  • Add new YAML-related dependencies


Changes walkthrough 📝

Relevant files
Configuration changes
flake.nix
Update PostgreSQL client version                                                 

flake.nix

  • Update PostgreSQL client from version 16.4 to 17.2
+1/-1     
Dependencies
go.mod
Update Go version and dependencies                                             

go.mod

  • Update Go version to 1.23.0
  • Upgrade multiple dependencies to latest versions
  • Add new dependencies for YAML handling
  • +30/-30 
    go.sum
    Update dependency checksums                                                           

    go.sum

  • Update checksums for upgraded dependencies
  • Remove old dependency versions
  • Add checksums for new dependencies
  • +62/-66 

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • Copy link
    Contributor

    github-actions bot commented Feb 3, 2025

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

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

    Dependency Updates

    Multiple dependencies have been updated to newer versions. Ensure compatibility and test for any breaking changes.

    go 1.23.0
    
    toolchain go1.23.1
    
    require (
    	github.com/MicahParks/keyfunc/v3 v3.3.10
    	github.com/bradfitz/gomemcache v0.0.0-20230905024940-24af94b03874
    	github.com/getkin/kin-openapi v0.129.0
    	github.com/gin-gonic/gin v1.10.0
    	github.com/go-webauthn/webauthn v0.11.2
    	github.com/gobwas/glob v0.2.3
    	github.com/golang-jwt/jwt/v5 v5.2.1
    	github.com/google/go-cmp v0.6.0
    	github.com/google/uuid v1.6.0
    	github.com/jackc/pgx/v5 v5.7.2
    	github.com/lmittmann/tint v1.0.7
    	github.com/oapi-codegen/gin-middleware v1.0.2
    	github.com/oapi-codegen/runtime v1.1.1
    	github.com/urfave/cli/v2 v2.27.5
    	github.com/valyala/fasttemplate v1.2.2
    	go.uber.org/mock v0.5.0
    	golang.org/x/crypto v0.32.0
    	k8s.io/client-go v0.32.1
    )
    PostgreSQL Version Change

    PostgreSQL client version has been updated from 16.4 to 17.2. Verify compatibility with the current database setup and any related configurations.

    postgresql_17_2-client

    Copy link
    Contributor

    github-actions bot commented Feb 3, 2025

    PR Code Suggestions ✨

    Explore these optional code suggestions:

    CategorySuggestion                                                                                                                                    Score
    General
    Update to latest stable Go version

    Consider updating the Go version to the latest stable release for improved
    performance and security. Currently, the PR is using Go 1.23.0, which is not a valid
    Go version.

    go.mod [3]

    -go 1.23.0
    +go 1.21.5
    Suggestion importance[1-10]: 9

    Why: The suggestion correctly identifies that Go 1.23.0 is not a valid version and recommends updating to a stable release. This is crucial for ensuring compatibility, security, and access to the latest features.

    9
    Align toolchain with Go version

    Ensure that the toolchain directive is consistent with the go directive. Currently,
    they specify different versions which may lead to confusion or potential issues.

    go.mod [3-5]

    -go 1.23.0
    +go 1.21.5
     
    -toolchain go1.23.1
    +toolchain go1.21.5
    Suggestion importance[1-10]: 8

    Why: The suggestion rightly points out the inconsistency between the go directive and the toolchain directive. Aligning these versions is important for maintaining a consistent development environment and avoiding potential conflicts.

    8

    @dbarrosop dbarrosop merged commit 1495588 into main Feb 3, 2025
    6 checks passed
    @dbarrosop dbarrosop deleted the update-deps branch February 3, 2025 15:09
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    2 participants