AYS-573 | SonarQube Has Been Integrated to GitHub Actions

[plnknr]

Checklist

Before submitting your pull request, ensure the following:

• Change Type:

• Bug fix

• Feature addition

• Documentation update

• Performance improvement

• Other (please specify)

• Summary:

  • Change quality-gate setting in Sonarqube integration fe-landing

• Title and Branch Naming Conventions:

  • The pull request title follows the
    standard: Pull Request Naming Conventions.
  • The branch name follows one of the conventions outlined in
    the Branch Naming Conventions.

• Local Testing:

  • I have tested my changes locally on my browser, and they are working as expected.

• Code Quality:

  • The code is formatted according to the project's coding guidelines and style.
  • The code has been reviewed to ensure its quality.
  • The code does not contain any issues flagged by ESLint.

• Documentation:

  • Necessary documentation has been added or existing documentation has been updated.

• Reviewers and Assignees:

  • Default reviewers have been assigned to this pull request.
  • Assignees have been added if necessary.

• Related Issue:

  • Closes #AYS-573

• Labels and Associations:

  • No specific actions are required in the Labels and Associations section for this pull request.

[sanshigo345]

bu pr ile ne fixlenmek istendi? bu bir fix mi? branch ismi feature ile başlıyor fix ile bitiyor ama PR checklistte feature seçilmiş tam anlamadım.

[yusufkaya01]

Bu haliyle şu an merge lenemez. FE de coverage olmadığı için pipeline hata verecek default QG de. Diğer repo için de geçerli. Ya sonar tarafında yeni bir QG yapılması gerekiyor ya da bu kısmın silinmesi gerekiyor. Ben FE için ayrı QG kullanılması taraftarıyım.

[yusufkaya01]

Zaten actions hata vermiş.

[yusufkaya01]

necessary lines has changed in workflows and QG on saonrqube for FE repos also changed

[sanshigo345]

bu PR ile yapılmak istenen nedir?

[agitrubard]

@plnknr pull request başlığını standartlara uygun olacak şekilde düzenleyebilir miyiz?

[plnknr]

bu PR ile yapılmak istenen nedir?

burada Quality Gate ayarları ile alakalı kısımları güncelledik.

[plnknr]

@plnknr pull request başlığını standartlara uygun olacak şekilde düzenleyebilir miyiz?

düzelttim umarım uygun olmuştur

[sanshigo345]

#87 (review)

#87 (review)

bunlara cevap alabilir miyim?

[sanshigo345]

bu PR ile yapılmak istenen nedir?

burada Quality Gate ayarları ile alakalı kısımları güncelledik.

Quality Gate'in ayarları neler oldu? Quality Gate conditionlara hangi ekip karar verdi? Quality Gate eklenmesi AYS-573 ile alakalı mı?

[plnknr]

bu pr ile ne fixlenmek istendi? bu bir fix mi? branch ismi feature ile başlıyor fix ile bitiyor ama PR checklistte feature seçilmiş tam anlamadım.

aslında fix adını kendi yapmam gereken kısmı düzeltme yaptım demek amacı ile yazmıştım bug fix kısmı ile alakası yoktu özellik ekleme kısmı ile alakalıydı. Benim hatam kendime not düştüm ama yanlış anlaşıldı affedersiniz :)

[plnknr]

bu PR ile yapılmak istenen nedir?

burada Quality Gate ayarları ile alakalı kısımları güncelledik.

Quality Gate'in ayarları neler oldu? Quality Gate conditionlara hangi ekip karar verdi? Quality Gate eklenmesi AYS-573 ile alakalı mı?

1-Quality Gate ayarlarını açtık ve quality gate kontrolünden (kodun kalite standartlarında olduğundan emin olmak için kullanılan ve hatalı veya eksik kod gibi problemleri önceden bildiren bir ayar) geçerse pipeline'a devam eder. Eğer geçemez ise pipeline'a devam etmez. Deployment kısmını gerçekleştirmez 2-Bu conditionlar'a ekip olarak hep beraber karar verilmesi doğru kısım ancak back-end kısmında sonarqube'ün bize verdiği kodda bu ayarı bize yaptırmıyor sanırım kendi içerisinde hallediyor ancak maven .net grandle dışındaki kullanımlar için bu ayarı bize sunuyor yazılım kalitesinin sürekliliğini sağlamak amacıyla kullandığımız için yararlı olabileceğini düşündük. 3- Evet Quality Gate eklenmesi AYS-573 ile alakalıydı bu entegrasyonda bize verilen kod içerisinde mevcuttu. Umarım açıklayıcı yazabilmişimdir :)

[plnknr]

bu PR ile yapılmak istenen nedir?

Bunun ile Quality Gate ayarlarını açmak ve aktif hale getirmek

[yusufkaya01]

@sanshigo345 FE repolarının QG leri sonar üzerinde değişti. Pipeline ın fail olmaması için coverage ı 0 olan FE repolarına ait yeni bir QG oluşturuldu. FE kod standartları için ihtiyaç duyulacak güncellemeleri sonar üzerinde yapacağız. Workflow a QG yi eklememizin sebebi .yaml dosyalarına daha sonra müdahale etmemek için.

[sanshigo345]

bu PR ile yapılmak istenen nedir?

burada Quality Gate ayarları ile alakalı kısımları güncelledik.

Quality Gate'in ayarları neler oldu? Quality Gate conditionlara hangi ekip karar verdi? Quality Gate eklenmesi AYS-573 ile alakalı mı?

1-Quality Gate ayarlarını açtık ve quality gate kontrolünden (kodun kalite standartlarında olduğundan emin olmak için kullanılan ve hatalı veya eksik kod gibi problemleri önceden bildiren bir ayar) geçerse pipeline'a devam eder. Eğer geçemez ise pipeline'a devam etmez. Deployment kısmını gerçekleştirmez 2-Bu conditionlar'a ekip olarak hep beraber karar verilmesi doğru kısım ancak back-end kısmında sonarqube'ün bize verdiği kodda bu ayarı bize yaptırmıyor sanırım kendi içerisinde hallediyor ancak maven .net grandle dışındaki kullanımlar için bu ayarı bize sunuyor yazılım kalitesinin sürekliliğini sağlamak amacıyla kullandığımız için yararlı olabileceğini düşündük. 3- Evet Quality Gate eklenmesi AYS-573 ile alakalıydı bu entegrasyonda bize verilen kod içerisinde mevcuttu. Umarım açıklayıcı yazabilmişimdir :)

gayet açıklayıcı yazmışsın, teşekkür ederim.

"Bu conditionlar'a ekip olarak hep beraber karar verilmesi doğru kısım" ise doğrusunu yapalım? özellikle FE reposunun pipelarını durduracağı için, en çok FE ekibini etkileyeceği için, FE ekibinin karar verdiği ayarların kullanılmasının mantıklı olacağını düşünüyorum.

"Quality Gate'in ayarları neler oldu?" diye sorarken ayarların neler olduğunu sormak istemiştim. Quality Gate ayarları ne demek diye sormak istememiştim. Özellikle FE pipelineları durdurucak olan bu ayarları bari FE ekibi bilsin?

Quality Gate'in eklenmesi AYS-573 ile alakalıysa lütfen AYS-573 ticketını geç de olsa ona göre düzenleyelim. (Quality Gate ayarlarının seçileceği, Quality gate'in aktifleştirileceği, seçilen ayarların belgeleneceği, buna göre repo içindeki gerekli belgelerin güncelleneceği vs gibi isterleri de ticket'a ekleyelim) Ayrıca AYS-573 için açık olan bir PR varken ticketı da Kanbanda "in review" kısmına taşımak mantıklı olacaktır.

"Bize verilen kod içerisinde mevcuttu derken" tam olarak hangi koddan bahsediliyor? Sonarqube entegrasyonu bence gayet Quality Gate pipelineları patlatmayacak şekilde de yapılabilir diye düşünüyorum. (ki yapılmış ve onaylanmıştı da)

[yusufkaya01]

Şu an için FE repolarında coverage olmadığı için FE QG leri 0% coverage arıyor. Pipeline ın patlaması için ise %3 duplication olması gerekiyor. Pipeline ın OK olması geçer kriterler şimdilik bunlar. Gerekli duyulan ekstra ayarlamaları FE & DevOps ekipleri olarak ayrıca görüşebiliriz. Gerek duyulduğu takdirde bunun için bir toplantı tarihi belirleyebiliriz.

[sanshigo345]

@sanshigo345 FE repolarının QG leri sonar üzerinde değişti. Pipeline ın fail olmaması için coverage ı 0 olan FE repolarına ait yeni bir QG oluşturuldu. FE kod standartları için ihtiyaç duyulacak güncellemeleri sonar üzerinde yapacağız. Workflow a QG yi eklememizin sebebi .yaml dosyalarına daha sonra müdahale etmemek için.

Şu an için FE repolarında coverage olmadığı için FE QG leri 0% coverage arıyor. Pipeline ın patlaması için ise %3 duplication olması gerekiyor. Pipeline ın OK olması geçer kriterler şimdilik bunlar. Gerekli duyulan ekstra ayarlamaları FE & DevOps ekipleri olarak ayrıca görüşebiliriz. Gerek duyulduğu takdirde bunun için bir toplantı tarihi belirleyebiliriz.

ok, DevOps ekibi yada siz ( @plnknr @yusufkaya01 ) FE ekibine sonarqube nedir ne değildir, quality gate nedir, quality gate ayarları FE projesi için neler seçilebilir, neden gereklidir, değildir vs içerikli bir bilgilendirme toplantısı yapsın. FE ekibi Quality Gate isteyip istemediğine, istiyorsa hangi standartları, ayarları istediğine karar versin. Quality Gate FE ekibi tarafından isteniyorsa, FE ekibinin kararlaştırdığı, istenilen ayarlar DevOps tarafından Sonarqube'e eklensin. Bu eklemelere göre repodaki dökümanlar güncellensin (mesela PR checklistte codequality kısmına sonarqube'ün sıkıntı çıkarmadığı eklensin, sonarqube hata verince PR sahibinin ne yapması gerekiyor yazılsın gibi gibi)

Bu işlemler tamamlandıktan sonra artık müdahale gerekmeyen .yaml dosyalarını içeren bu PR'ı mergeleyelim 👍

[plnknr]

@sanshigo345 FE repolarının QG leri sonar üzerinde değişti. Pipeline ın fail olmaması için coverage ı 0 olan FE repolarına ait yeni bir QG oluşturuldu. FE kod standartları için ihtiyaç duyulacak güncellemeleri sonar üzerinde yapacağız. Workflow a QG yi eklememizin sebebi .yaml dosyalarına daha sonra müdahale etmemek için.

Şu an için FE repolarında coverage olmadığı için FE QG leri 0% coverage arıyor. Pipeline ın patlaması için ise %3 duplication olması gerekiyor. Pipeline ın OK olması geçer kriterler şimdilik bunlar. Gerekli duyulan ekstra ayarlamaları FE & DevOps ekipleri olarak ayrıca görüşebiliriz. Gerek duyulduğu takdirde bunun için bir toplantı tarihi belirleyebiliriz.

ok, DevOps ekibi yada siz ( @plnknr @yusufkaya01 ) FE ekibine sonarqube nedir ne değildir, quality gate nedir, quality gate ayarları FE projesi için neler seçilebilir, neden gereklidir, değildir vs içerikli bir bilgilendirme toplantısı yapsın. FE ekibi Quality Gate isteyip istemediğine, istiyorsa hangi standartları, ayarları istediğine karar versin. Quality Gate FE ekibi tarafından isteniyorsa, FE ekibinin kararlaştırdığı, istenilen ayarlar DevOps tarafından Sonarqube'e eklensin. Bu eklemelere göre repodaki dökümanlar güncellensin (mesela PR checklistte codequality kısmına sonarqube'ün sıkıntı çıkarmadığı eklensin, sonarqube hata verince PR sahibinin ne yapması gerekiyor yazılsın gibi gibi)

Bu işlemler tamamlandıktan sonra artık müdahale gerekmeyen .yaml dosyalarını içeren bu PR'ı mergeleyelim 👍

Tamamdır Buğra böyle daha iyi olacak diyorsanız bizim içinde uygundur bununla alakalı planlamayı slack üzerinden ayrıntılı konuşarak yapabiliriz

[agitrubard]

@plnknr actions/checkout@v3 kısmını da actions/checkout@v4 olarak güncelleyebilir miyiz?