Skip to content

Conversation

@vishal210893
Copy link
Contributor

@vishal210893 vishal210893 commented Jun 5, 2025

Summary by cubic

Upgraded the Go version to 1.22.5 and updated kin-openapi and related dependencies for better compatibility and support.

  • Dependencies
    • Updated kin-openapi to v0.131.0 and related packages.
    • Changed the Parameter.JSONType field to use *openapi3.Types.
    • Updated other indirect dependencies to match new requirements.

@vishal210893 vishal210893 force-pushed the feature/upgrade-version branch from b3c8d97 to 2ef31ba Compare June 5, 2025 17:57
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

cubic found 2 issues across 2 files. Review them in cubic.dev

React with 👍 or 👎 to teach cubic. Tag @cubic-dev-ai to give specific feedback.

go 1.19
go 1.22.5

toolchain go1.24.2
Copy link

Choose a reason for hiding this comment

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

The toolchain directive should reference a Go toolchain in major.minor form; including a patch suffix makes the directive invalid and prevents module commands from running.

Suggested change
toolchain go1.24.2
toolchain go1.24

module github.com/oam-dev/kubevela-core-api

go 1.19
go 1.22.5
Copy link

Choose a reason for hiding this comment

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

The go directive must specify only the language version (major.minor). Including the patch number results in an invalid go.mod file and will break go commands.

Suggested change
go 1.22.5
go 1.22

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.

2 participants