Skip to content

Comments

[Automated] Sync master after 2.17.0 release#1448

Open
ballerina-bot wants to merge 3 commits intomasterfrom
release-2.17.0
Open

[Automated] Sync master after 2.17.0 release#1448
ballerina-bot wants to merge 3 commits intomasterfrom
release-2.17.0

Conversation

@ballerina-bot
Copy link
Contributor

@ballerina-bot ballerina-bot commented Feb 20, 2026

Sync master after 2.17.0 release

This pull request syncs the master branch following the 2.17.0 release. The changes finalize the release by updating platform dependencies from SNAPSHOT builds to stable release versions and prepare the codebase for the next development cycle.

Key Changes:

  • Updated three core platform dependencies in ballerina/Ballerina.toml from version 2.17.0-SNAPSHOT to the final 2.17.0 release:

    • log-native
    • log-compiler-plugin
    • log-test-utils
  • Updated the compiler plugin dependency in ballerina/CompilerPlugin.toml from 2.17.0-SNAPSHOT to 2.17.0

  • Incremented the project version in gradle.properties from 2.17.0-SNAPSHOT to 2.17.1-SNAPSHOT to begin the next development cycle

These are routine, automated post-release synchronization updates that align dependencies with the stable release and prepare the repository for ongoing development.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 20, 2026

📝 Walkthrough

Walkthrough

The pull request updates three platform dependencies in configuration files from SNAPSHOT to non-SNAPSHOT release versions, and increments the Gradle project version from 2.17.0-SNAPSHOT to 2.17.1-SNAPSHOT. No logic or control flow changes are introduced.

Changes

Cohort / File(s) Summary
Dependency snapshot-to-release updates
ballerina/Ballerina.toml, ballerina/CompilerPlugin.toml
Updated log library dependencies (log-native, log-compiler-plugin, log-test-utils) from SNAPSHOT jar paths to non-SNAPSHOT stable release paths.
Version bump
gradle.properties
Incremented project version from 2.17.0-SNAPSHOT to 2.17.1-SNAPSHOT.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

🐰 A hop and a skip, our snapshots take flight,
Release jars gleaming, no more -SNAPSHOT in sight,
Version bumped up, stability's here,
Building on solid ground, crystal clear! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is minimal and only contains a brief statement. It lacks the required template sections (Purpose, Examples, Checklist) that should be completed according to the repository template. Expand the description to follow the template structure: add Purpose section explaining the release sync, Examples if applicable, and complete the Checklist items (linked issue, changelog, tests, spec, native-image compatibility).
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically summarizes the main change - syncing the master branch after the 2.17.0 release, which aligns with the changeset updates to dependencies and version numbers.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch release-2.17.0

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
ballerina/Ballerina.toml (1)

15-31: ⚠️ Potential issue | 🔴 Critical

Jar filenames in Ballerina.toml will not match build outputs.
The project version is 2.17.1-SNAPSHOT (gradle.properties). The build template (build-config/resources/Ballerina.toml) uses @project.version@ placeholders, which expand to 2.17.1-SNAPSHOT, producing jars like log-native-2.17.1-SNAPSHOT.jar. However, ballerina/Ballerina.toml hardcodes paths referencing 2.17.0.jar jars. This mismatch will break packaging and plugin resolution.

Either use the templating system with @project.version@ placeholders, or ensure the hardcoded versions (2.17.0) are built and copied explicitly.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@ballerina/Ballerina.toml` around lines 15 - 31, The Ballerina.toml currently
hardcodes dependency paths and filenames with version "2.17.0" (entries for
artifactId "log-native", "log-compiler-plugin", "log-test-utils"), which will
not match the actual build version (2.17.1-SNAPSHOT); update those dependency
blocks to use the templating placeholder for the project version (replace the
"2.17.0" literal in both the version and the path filenames with
`@project.version`@) so the generated jar names (e.g.,
log-native-@project.version@.jar) match the build outputs, or alternatively
change `@project.version`@ in the build template to 2.17.0 if you intend to keep
building 2.17.0 — ensure the artifactId strings ("log-native",
"log-compiler-plugin", "log-test-utils") remain unchanged while making the
version/path dynamic and consistent with gradle.properties.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@ballerina/Ballerina.toml`:
- Around line 15-31: The Ballerina.toml currently hardcodes dependency paths and
filenames with version "2.17.0" (entries for artifactId "log-native",
"log-compiler-plugin", "log-test-utils"), which will not match the actual build
version (2.17.1-SNAPSHOT); update those dependency blocks to use the templating
placeholder for the project version (replace the "2.17.0" literal in both the
version and the path filenames with `@project.version`@) so the generated jar
names (e.g., log-native-@project.version@.jar) match the build outputs, or
alternatively change `@project.version`@ in the build template to 2.17.0 if you
intend to keep building 2.17.0 — ensure the artifactId strings ("log-native",
"log-compiler-plugin", "log-test-utils") remain unchanged while making the
version/path dynamic and consistent with gradle.properties.

@codecov
Copy link

codecov bot commented Feb 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.66%. Comparing base (53a45a4) to head (751abbd).

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1448   +/-   ##
=========================================
  Coverage     81.66%   81.66%           
  Complexity      108      108           
=========================================
  Files            10       10           
  Lines           731      731           
  Branches        156      156           
=========================================
  Hits            597      597           
  Misses           92       92           
  Partials         42       42           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant