Skip to content

Comments

Ensure predictable Ballerina dependency resolution in workflows#299

Draft
nipunayf wants to merge 4 commits intoballerina-platform:1.2.xfrom
nipunayf:fix-ballerina-dependencies
Draft

Ensure predictable Ballerina dependency resolution in workflows#299
nipunayf wants to merge 4 commits intoballerina-platform:1.2.xfrom
nipunayf:fix-ballerina-dependencies

Conversation

@nipunayf
Copy link
Contributor

@nipunayf nipunayf commented Sep 11, 2025

Purpose

Currently, the only way to lock transitive dependency versions is by specifying the required version in Ballerina.toml and building with the --sticky flag. This PR updates the existing usage of bal pull to follow this approach.

Fixes wso2/product-ballerina-integrator#1245

Copilot AI review requested due to automatic review settings September 11, 2025 08:59
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 ensures predictable Ballerina dependency resolution by replacing the use of bal pull with a Ballerina.toml file specifying dependency versions with the --sticky flag for builds.

  • Removes version definitions from gradle.properties and individual module bal pull commands
  • Creates a centralized dependency management approach using Ballerina.toml with sticky builds
  • Temporarily disables Mistral AI provider tests due to dependency resolution issues

Reviewed Changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
gradle.properties Removes version variables for Ballerinax modules and fixes naming inconsistencies
build-config/ballerina_dependencies/Ballerina.toml Adds centralized dependency configuration with sticky build option
build-config/ballerina_dependencies/main.bal Creates placeholder file with imports for all required dependencies
flow-model-generator/modules/flow-model-generator-ls-extension/build.gradle Removes individual bal pull commands for modules
GetNodeTemplateTest.java Comments out Mistral test case with tracking reference
CodeGenerationTest.java Comments out Mistral test case with tracking reference
architecture-model-generator/modules/architecture-model-generator-ls-extension/build.gradle Adds --sticky flag to tool pull commands

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

@nipunayf nipunayf changed the title Ensure predictable Ballerina dependency resolution in workflows. Ensure predictable Ballerina dependency resolution in workflows Sep 11, 2025
@nipunayf nipunayf marked this pull request as draft September 11, 2025 09:01
@nipunayf nipunayf force-pushed the fix-ballerina-dependencies branch from 52b2b17 to 1ed96e4 Compare September 11, 2025 09:33
@nipunayf nipunayf force-pushed the fix-ballerina-dependencies branch 2 times, most recently from 73115bd to e62f2f2 Compare September 11, 2025 13:53
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