AYS-572 | 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-institution

• 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-572

• Labels and Associations:

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

[plnknr]

landing kısmındaki güncellemeler burada da yapıldı.

[agitrubard]

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

[sanshigo345]

afet-yonetim-sistemi/ays-fe-landing#87 (comment)

aynısı geçerli

[agitrubard]

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

[plnknr]

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

@agitrubard güncelledim pull-request-checker kısmında da güncelliyorum landing reposu içinde güncelliyorum.

[agitrubard]

@yusufkaya01 @plnknr quality gate hata vermiş gibi görünüyor

[plnknr]

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

bu kısımda versiyonu neden güncelliyoruz ?

[plnknr]

@yusufkaya01 @plnknr quality gate hata vermiş gibi görünüyor

evet bunu araştırıyoruz. Mergelemekten oldu zannettik ama güncelleyince de düzelmedi.

[agitrubard]

bu kısımda versiyonu neden güncelliyoruz ?

@plnknr son versiyonu olduğu için oraya güncellememiz daha iyi olur, günceli takip etmek adına. Aksi durumda pipeline bozulabiliyor

[plnknr]

bu kısımda versiyonu neden güncelliyoruz ?

@plnknr son versiyonu olduğu için oraya güncellememiz daha iyi olur, günceli takip etmek adına. Aksi durumda pipeline bozulabiliyor

Tamamdır güncellemeleri landing ve institution workflowları için tamamladık. Ayrıca Quality Gate hatasını da şu şekilde çözdük; hatanın Sonarqube üzerinde ayarlanan issue condition'nın açık olmasından kaynaklı olduğunu tespit edip bu condition'ı kaldırdık böylece problem çözüldü @sanshigo345 @agitrubard

[sanshigo345]

bu kısımda versiyonu neden güncelliyoruz ?

@plnknr son versiyonu olduğu için oraya güncellememiz daha iyi olur, günceli takip etmek adına. Aksi durumda pipeline bozulabiliyor

Tamamdır güncellemeleri landing ve institution workflowları için tamamladık. Ayrıca Quality Gate hatasını da şu şekilde çözdük; hatanın Sonarqube üzerinde ayarlanan issue condition'nın açık olmasından kaynaklı olduğunu tespit edip bu condition'ı kaldırdık böylece problem çözüldü @sanshigo345 @agitrubard

bi ara birlikte bakalım @plnknr issue condition'ununu kapadık çözüldü mantıklı bir çözüm gibi gelmedi

[plnknr]

bu kısımda versiyonu neden güncelliyoruz ?

@plnknr son versiyonu olduğu için oraya güncellememiz daha iyi olur, günceli takip etmek adına. Aksi durumda pipeline bozulabiliyor

Tamamdır güncellemeleri landing ve institution workflowları için tamamladık. Ayrıca Quality Gate hatasını da şu şekilde çözdük; hatanın Sonarqube üzerinde ayarlanan issue condition'nın açık olmasından kaynaklı olduğunu tespit edip bu condition'ı kaldırdık böylece problem çözüldü @sanshigo345 @agitrubard

bi ara birlikte bakalım @plnknr issue condition'ununu kapadık çözüldü mantıklı bir çözüm gibi gelmedi

Tamamdır görüşme ayarlayalım bakalım

[yusufkaya01]

@sanshigo345 Sorunun kaynağını çözebilmek için issues condition'ı kapatmıştık, her ne kadar mantıklı(?) ve geçen konuştuğumuzda FE in isteği bu olmasa da. Ancak Sonar, yeni kodlarda issue bulduğu için QG'den geçemiyormuş. Bu PR'ı merge'leyip, FE'nin istediği şekilde 'yeni kodlarda issue olmasın, eklenen kodlar 0 issue ile merge'lensin' kısmını daha sonra halletmemiz gerekecek. PR'ı ne kadar geç merge edersek, sonraki aşamalarda repolardaki Sonar'dan gelen hataları o kadar kolay çözeriz. Şu an için pipeline düzgün ve stabil.

Ayrıca, repoların Sonar entegrasyonunu tamamlamamız lazım ki Sonar imajını güncel versiyonlarla takip edebilelim. Şu an eski bir versiyon kullanıyoruz ve bu da yeni versiyonlara göre daha yüksek vulnerability riski taşıyor.

Bu arada issues condition ı şu an kapalı olsa dahi sadece QG için kapalı, issue lar Sonar üzerinde analiz edilmeye devam ediyor.

[sanshigo345]

@sanshigo345 PR'ı ne kadar geç merge edersek, sonraki aşamalarda repolardaki Sonar'dan gelen hataları o kadar kolay çözeriz. ??

@plnknr @yusufkaya01 üçümüz birlikte konuşalım o zaman, hallederiz. müsait olduğumuz bir zaman kararlaştıralım

[yusufkaya01]

Konuyla ilgili olarak yaptığımız görüşme sonrasında; QG'nin eklenmesi için Jira da bir ticket oluşturup, aldığımız hatanın giderilmesi ve ileride aynı şekilde bir sorun olmamasının önüne nasıl geçebileceğimizi öğrenme konusunda fikir birliği sağladık. Böylece QG'in FE repolarına eklenmesini ertelemiş olduk.

QG'in workflow a entegre edilmesi konusu benim tarafımdan önerildiği ve biraz karışıklığa neden olmamdandan dolayı özür diliyorum :) Herkese iyi çalışmalar.