Skip to content

Conversation

@javuto
Copy link
Collaborator

@javuto javuto commented Dec 24, 2025

Dependencies update and fixes for CICD pipeline errors

@javuto javuto requested a review from Copilot December 24, 2025 09:51
@javuto javuto added dependencies Dependencies related issues cicd CICD related issues labels Dec 24, 2025
Copy link
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 dependencies and fixes CI/CD pipeline issues. The changes primarily focus on bumping Go module versions and correcting the release workflow configuration.

  • Updates AWS SDK, Elasticsearch, OpenTelemetry, and other Go dependencies to newer versions
  • Removes --skip-publish flag from goreleaser in release workflow
  • Adds binary download step in tagged releases workflow

Reviewed changes

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

File Description
go.mod Updates multiple dependencies including AWS SDK v2, Elasticsearch, golang.org/x packages, and promotes go.yaml.in/yaml/v2 from indirect to direct dependency
.github/workflows/release.yml Removes --skip-publish flag from goreleaser arguments to enable artifact publishing
.github/workflows/create_tagged_releases.yml Adds download-artifact step to retrieve binaries before uploading release assets
Comments suppressed due to low confidence (1)

go.mod:3

  • Go version 1.25.4 does not exist. As of January 2025, the latest stable Go version is 1.23.x. This should be corrected to a valid Go version.
go 1.25.4

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

@javuto javuto merged commit dec9154 into main Dec 24, 2025
55 checks passed
@javuto javuto deleted the cicd-fixes-deps-update branch December 24, 2025 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cicd CICD related issues dependencies Dependencies related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants