[Automated] Sync master after 1.18.0 release#788
[Automated] Sync master after 1.18.0 release#788ballerina-bot wants to merge 5 commits intomasterfrom
Conversation
ballerina-bot
requested review from
RDPerera,
niveathika and
shafreenAnfar
as code owners
📝 WalkthroughWalkthroughConfiguration and version updates across multiple build files. Dependency paths in platform configurations were updated from SNAPSHOT to release JARs, package versions in Dependencies.toml were downgraded, and the Gradle project version was incremented to a new SNAPSHOT version. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (1)
ballerina/CompilerPlugin.toml (1)
6-6: Same JAR path /gradle.propertiesversion mismatch asBallerina.toml.
gradle.propertiesis at1.18.1-SNAPSHOT, so Gradle will producesql-compiler-plugin-1.18.1-SNAPSHOT.jar, while this path still points tosql-compiler-plugin-1.18.0.jar. See the comment onBallerina.tomllines 19–22 for details; both paths will need updating to1.18.1-SNAPSHOT.jarbefore a clean build from master succeeds.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@ballerina/CompilerPlugin.toml` at line 6, The CompilerPlugin.toml "path" entry references sql-compiler-plugin-1.18.0.jar which mismatches the gradle.properties version (1.18.1-SNAPSHOT); update the "path" value in CompilerPlugin.toml to sql-compiler-plugin-1.18.1-SNAPSHOT.jar and make the same change to the corresponding artifact reference in Ballerina.toml so both manifest entries match the gradle.properties version (1.18.1-SNAPSHOT).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@ballerina/CompilerPlugin.toml`:
- Line 6: Update the jar path in CompilerPlugin.toml so it matches the Gradle
artifact version used (change the path string "path =
\"../compiler-plugin/build/libs/sql-compiler-plugin-1.18.0.jar\"" to the
artifact produced by Gradle, e.g. "sql-compiler-plugin-1.18.1-SNAPSHOT.jar" if
gradle.properties uses 1.18.1-SNAPSHOT); ensure the path value in
CompilerPlugin.toml is consistent with the version declared in gradle.properties
and the related Ballerina.toml entries so all three reference the same jar
filename.
---
Duplicate comments:
In `@ballerina/CompilerPlugin.toml`:
- Line 6: The CompilerPlugin.toml "path" entry references
sql-compiler-plugin-1.18.0.jar which mismatches the gradle.properties version
(1.18.1-SNAPSHOT); update the "path" value in CompilerPlugin.toml to
sql-compiler-plugin-1.18.1-SNAPSHOT.jar and make the same change to the
corresponding artifact reference in Ballerina.toml so both manifest entries
match the gradle.properties version (1.18.1-SNAPSHOT).
ℹ️ Review info
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
ballerina/Ballerina.tomlballerina/CompilerPlugin.tomlballerina/Dependencies.tomlgradle.properties
|
|
||
| [[dependency]] | ||
| path = "../compiler-plugin/build/libs/sql-compiler-plugin-1.18.0-SNAPSHOT.jar" | ||
| path = "../compiler-plugin/build/libs/sql-compiler-plugin-1.18.0.jar" |
There was a problem hiding this comment.
Same JAR path / gradle.properties mismatch as Ballerina.toml lines 19–22.
Gradle at 1.18.1-SNAPSHOT will produce sql-compiler-plugin-1.18.1-SNAPSHOT.jar, but this path still declares sql-compiler-plugin-1.18.0.jar. The same follow-up path-bump PR that fixes Ballerina.toml should address this entry as well.
🔧 Expected fix
-path = "../compiler-plugin/build/libs/sql-compiler-plugin-1.18.0.jar"
+path = "../compiler-plugin/build/libs/sql-compiler-plugin-1.18.1-SNAPSHOT.jar"📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| path = "../compiler-plugin/build/libs/sql-compiler-plugin-1.18.0.jar" | |
| path = "../compiler-plugin/build/libs/sql-compiler-plugin-1.18.1-SNAPSHOT.jar" |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@ballerina/CompilerPlugin.toml` at line 6, Update the jar path in
CompilerPlugin.toml so it matches the Gradle artifact version used (change the
path string "path =
\"../compiler-plugin/build/libs/sql-compiler-plugin-1.18.0.jar\"" to the
artifact produced by Gradle, e.g. "sql-compiler-plugin-1.18.1-SNAPSHOT.jar" if
gradle.properties uses 1.18.1-SNAPSHOT); ensure the path value in
CompilerPlugin.toml is consistent with the version declared in gradle.properties
and the related Ballerina.toml entries so all three reference the same jar
filename.
Codecov Report✅ All modified and coverable lines are covered by tests. ❌ Your project check has failed because the head coverage (78.11%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## master #788 +/- ##
=========================================
Coverage 78.11% 78.11%
Complexity 109 109
=========================================
Files 41 41
Lines 3761 3761
Branches 618 618
=========================================
Hits 2938 2938
Misses 565 565
Partials 258 258 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
1 participant
Sync master after 1.18.0 release
Release Synchronization: 1.18.0 Version Finalization
This PR synchronizes the master branch following the completion of the 1.18.0 release. It finalizes the release artifacts and prepares the codebase for continued development.
Key Changes
Release Artifact Finalization:
ballerina/Ballerina.tomlfrom SNAPSHOT versions to final release versions forsql-nativeandsql-test-utilsdependenciesballerina/CompilerPlugin.tomlfrom SNAPSHOT to final release buildDevelopment Version Update:
gradle.propertiesfrom1.18.0-SNAPSHOTto1.18.1-SNAPSHOT, establishing the next development cycleDependency Version Adjustments:
ballerina/Dependencies.tomlfor crypto, http, log, and observe modulesThese changes are typical of post-release synchronization, removing temporary snapshot designations from production artifacts while preparing the codebase for continued feature development under the next version.