Skip to content

Conversation

@milindl
Copy link
Contributor

@milindl milindl commented Sep 22, 2025

Copilot AI review requested due to automatic review settings September 22, 2025 16:22
@milindl milindl requested review from a team as code owners September 22, 2025 16:22
Copy link

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 bumps the version from 1.4.1 to 1.5.0-rc1-dev across the project, preparing for a pre-release candidate build.

  • Updates version numbers in package.json files and utility module
  • Adds version mapping entry in README documentation
  • Maintains consistency across all version references in the codebase

Reviewed Changes

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

File Description
package.json Updates main package version to 1.5.0-rc1-dev
schemaregistry/package.json Updates schema registry package version to 1.5.0-rc1-dev
lib/util.js Updates binding version string to match new version
README.md Adds version mapping entry for 1.5.0 release

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

package.json Outdated
{
"name": "@confluentinc/kafka-javascript",
"version": "1.4.1",
"version": "1.5.0-rc1-dev",
Copy link

Copilot AI Sep 22, 2025

Choose a reason for hiding this comment

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

The version format '1.5.0-rc1-dev' is non-standard. According to semantic versioning, pre-release versions should use '-' for pre-release identifiers but avoid mixing 'rc' and 'dev' suffixes. Consider using either '1.5.0-rc.1' or '1.5.0-dev' format.

Suggested change
"version": "1.5.0-rc1-dev",
"version": "1.5.0-rc.1",

Copilot uses AI. Check for mistakes.
{
"name": "@confluentinc/schemaregistry",
"version": "1.4.1",
"version": "1.5.0-rc1-dev",
Copy link

Copilot AI Sep 22, 2025

Choose a reason for hiding this comment

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

The version format '1.5.0-rc1-dev' is non-standard. According to semantic versioning, pre-release versions should use '-' for pre-release identifiers but avoid mixing 'rc' and 'dev' suffixes. Consider using either '1.5.0-rc.1' or '1.5.0-dev' format.

Suggested change
"version": "1.5.0-rc1-dev",
"version": "1.5.0-rc.1",

Copilot uses AI. Check for mistakes.
lib/util.js Outdated
};

util.bindingVersion = '1.4.1';
util.bindingVersion = '1.5.0-rc1-dev';
Copy link

Copilot AI Sep 22, 2025

Choose a reason for hiding this comment

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

The version format '1.5.0-rc1-dev' is non-standard. According to semantic versioning, pre-release versions should use '-' for pre-release identifiers but avoid mixing 'rc' and 'dev' suffixes. Consider using either '1.5.0-rc.1' or '1.5.0-dev' format.

Suggested change
util.bindingVersion = '1.5.0-rc1-dev';
util.bindingVersion = '1.5.0-rc.1';

Copilot uses AI. Check for mistakes.
@sonarqube-confluent

This comment has been minimized.

@sonarqube-confluent
Copy link

Passed

Analysis Details

0 Issues

  • Bug 0 Bugs
  • Vulnerability 0 Vulnerabilities
  • Code Smell 0 Code Smells

Coverage and Duplications

  • Coverage 100.00% Coverage (70.90% Estimated after merge)
  • Duplications No duplication information (2.00% Estimated after merge)

Project ID: confluent-kafka-javascript

View in SonarQube

@milindl milindl merged commit 348266f into master Sep 23, 2025
3 checks passed
@milindl milindl deleted the dev_update_1.5.0-release-candidate-1 branch September 23, 2025 05:42
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