awasum commented on a change in pull request #1955:
URL: https://github.com/apache/fineract/pull/1955#discussion_r743186530



##########
File path: .github/workflows/build-docker.yml
##########
@@ -0,0 +1,20 @@
+name: Fineract Docker build
+
+on: [push]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest

Review comment:
       Lets use Latest LTS version say Ubuntu 20.04 here..Minor..but makes 
sense. See: 
https://cloudmaniac.net/docker-container-never-use-latest-cicd-pipeline/

##########
File path: .github/workflows/build.yml
##########
@@ -0,0 +1,38 @@
+name: Fineract Gradle build
+on: [push]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    env:
+        TZ: Asia/Kolkata

Review comment:
       @ptuomola  already answered this here and it makes sense. Thanks...so 
resolving this. see: 
https://github.com/apache/fineract/pull/1955#issuecomment-961172334

##########
File path: .github/workflows/sonarqube.yml
##########
@@ -0,0 +1,32 @@
+name: Fineract Sonarqube
+on: [push]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest

Review comment:
       Just minor: See above: 
https://github.com/apache/fineract/pull/1955/files#r743186530

##########
File path: .github/workflows/build-docker.yml
##########
@@ -0,0 +1,20 @@
+name: Fineract Docker build
+
+on: [push]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest

Review comment:
       Lets use Latest LTS version say Ubuntu 20.04 here..Minor..but makes 
sense. See: 
https://cloudmaniac.net/docker-container-never-use-latest-cicd-pipeline/

##########
File path: .github/workflows/build.yml
##########
@@ -0,0 +1,38 @@
+name: Fineract Gradle build
+on: [push]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    env:
+        TZ: Asia/Kolkata

Review comment:
       @ptuomola  already answered this here and it makes sense. Thanks...so 
resolving this. see: 
https://github.com/apache/fineract/pull/1955#issuecomment-961172334

##########
File path: .github/workflows/sonarqube.yml
##########
@@ -0,0 +1,32 @@
+name: Fineract Sonarqube
+on: [push]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest

Review comment:
       Just minor: See above: 
https://github.com/apache/fineract/pull/1955/files#r743186530




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to