-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update build scripts #794
Update build scripts #794
Conversation
@@ -0,0 +1,122 @@ | |||
name: Build with Ballerina Lang |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need this workflow? Does the other ballerinax connectors have this workflow?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I got these workflows from the ballerinax/kafka
connector, since these have the compiler plugin I was asked to add the standard library workflows. https://github.com/ballerina-platform/module-ballerinax-kafka/blob/master/.github/workflows/build-with-ballerina-lang.yml
@@ -17,12 +17,12 @@ jobs: | |||
with: | |||
distribution: 'temurin' | |||
java-version: 17.0.7 | |||
|
|||
- name: Set environment variable |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is workflow is not centralized. Is this because this module has compiler plugin?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -23,15 +23,6 @@ plugins { | |||
id "net.researchgate.release" version "2.8.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shall we move the plugin management to the settings.gradle
file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this deleted?
Purpose
Fixes:
Examples
Checklist