Skip to content

Conversation

@mateeullahmalik
Copy link
Collaborator

No description provided.

@akobrin1 akobrin1 requested a review from Copilot October 23, 2025 13:49
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 upgrades the Lumera protocol dependency from v1.7.2 to v1.8.0 across the project, along with updating Go to version 1.25.1 and numerous transitive dependencies.

Key changes:

  • Upgrades Lumera from v1.7.2 to v1.8.0 across all modules
  • Updates Go version from 1.24.1 to 1.25.1
  • Refactors CascadeMetadata usage to use action types instead of API types with standard JSON marshaling
  • Fixes error handling in configuration verification to properly propagate errors

Reviewed Changes

Copilot reviewed 8 out of 12 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tests/system/go.mod Updates Go version, Lumera dependency, and numerous transitive dependencies
go.mod Updates Go version, Lumera dependency, Cosmos SDK, and transitive dependencies
cmd/sncli/go.mod Updates Go version, Lumera dependency, and transitive dependencies
supernode/verifier/verifier.go Fixes error return to properly propagate errors instead of returning nil
supernode/cmd/start.go Refactors error handling to combine error and validation result checks into unified logging
pkg/lumera/modules/action_msg/helpers.go Migrates from protojson marshaling with API types to standard JSON marshaling with action types
cmd/sncli/cli/cmd_get_status.go Updates field access from PascalCase to correct protobuf field names
Makefile Dynamically derives Lumera version from go.mod instead of hardcoded version

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@akobrin1 akobrin1 merged commit 7a83904 into master Oct 23, 2025
7 checks passed
@akobrin1 akobrin1 deleted the lumera-1.8.0 branch October 23, 2025 13:50
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