Skip to content

AB#530382: dependency updates - #116

Merged
Adam Cofer (acofer) merged 4 commits into
masterfrom
AB#530382-picard-veracode-remediation
Jun 26, 2026
Merged

AB#530382: dependency updates#116
Adam Cofer (acofer) merged 4 commits into
masterfrom
AB#530382-picard-veracode-remediation

Conversation

@njdullea

Copy link
Copy Markdown
Contributor

Issue Link

https://dev.azure.com/nintex/Nintex/_workitems/edit/530382

High-Level Description

Updates dependencies in Picard!

Changelog:

  • .github/workflows/ci.yml: Update go version to match go.mod.
  • .github/workflows/veracode-upload-and-scan.yml: Update go version to match go.mod.
  • go.mod: Update dependency version.
  • go.sum: Update package control.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Not ready to approve

The go.mod go directive is set to 1.25.0, which is not a valid go directive format for the Go toolchain and can break builds/CI.

Pull request overview

This PR updates the project’s Go toolchain configuration and bumps several Go module dependencies, aligning GitHub Actions workflows to derive the Go version directly from go.mod.

Changes:

  • Updated go.mod Go version and bumped selected direct/indirect dependencies (e.g., stretchr/testify, OpenTelemetry, x/*, gRPC/protobuf).
  • Refreshed go.sum to reflect the updated module graph/checksums.
  • Updated GitHub Actions workflows to use actions/setup-go@v5 with go-version-file: go.mod.
File summaries
File Description
go.mod Updates Go version and dependency versions.
go.sum Updates module checksums to match the new dependency graph.
.github/workflows/ci.yml Uses go-version-file so CI tracks the Go version declared in go.mod.
.github/workflows/veracode-upload-and-scan.yml Uses go-version-file so Veracode workflow tracks the Go version declared in go.mod.

Copilot's findings

  • Files reviewed: 3/4 changed files
  • Comments generated: 1

Note

Your feedback helps us improve the quality of this feature.
Please use 👍 or 👎 to tell us whether this assessment is correct.


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

Comment thread go.mod Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Not ready to approve

The Veracode workflow still uses the deprecated actions/checkout@v3, which should be updated to @v4 to avoid Node 16 deprecation breakage.

Copilot's findings
  • Files reviewed: 3/4 changed files
  • Comments generated: 1

Note

Your feedback helps us improve the quality of this feature.
Please use 👍 or 👎 to tell us whether this assessment is correct.

Comment thread .github/workflows/veracode-upload-and-scan.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

@elena-oneil Elena ONeil (elena-oneil) left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@acofer
Adam Cofer (acofer) merged commit 362a017 into master Jun 26, 2026
3 checks passed
@acofer
Adam Cofer (acofer) deleted the AB#530382-picard-veracode-remediation branch June 26, 2026 21:29
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.

4 participants