CodeForPhilly / clean-and-green-philly

Dashboard to target Philly vacant properties for anti-gun violence interventions
https://www.cleanandgreenphilly.org/
MIT License
32 stars 58 forks source link

DevOps: add SonarQube to the repo for code QC #751

Open nlebovits opened 1 month ago

nlebovits commented 1 month ago

Describe the task

We would like to integrate the community edition of SonarQube into the repository with GitHub actions. The goal is to improve our devops and reduce the load on our leads when reviewing PRs.

Acceptance Criteria

csharpie commented 1 month ago

Hello, I'd like to work on this issue. Please assign it to me.

nlebovits commented 1 month ago

@csharpie Assigned!

csharpie commented 1 month ago

Thanks Nissim.

Brad

On Wed, Jul 17, 2024 at 8:48 AM Nissim Lebovits @.***> wrote:

@csharpie https://github.com/csharpie Assigned!

— Reply to this email directly, view it on GitHub https://github.com/CodeForPhilly/clean-and-green-philly/issues/751#issuecomment-2233243267, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAPKRGWVMYDVT2KHRB6RTKDZMZR25AVCNFSM6AAAAABK7ABBGKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZTGI2DGMRWG4 . You are receiving this because you were mentioned.Message ID: @.***>

csharpie commented 1 month ago

I'll add a note on Slack too but it looks like SonarQube's Community Edition (because it's installed locally) doesn't support Pull Request Decorations. However there's an unofficial GitHub repository that does support this. I plan to look into this to see the feasibility.

CodeWritingCow commented 1 month ago

Hello @csharpie, any update on this ticket? We normally reassign the ticket if there's no activity on it for 1 week. I wanted to reach out before doing that. Thanks!

csharpie commented 1 month ago

Hey Gary,

I spent some time on it yesterday and made significant progress I think. I will update the issue today and let you know where I’ve left off.

Thanks for following up.

Sincerely, Brad Steinberg

On Mon, Aug 5, 2024 at 3:20 PM Gary Pang @.***> wrote:

Hello @csharpie https://github.com/csharpie, any update on this ticket? We normally reassign the ticket if there's no activity on it for 1 week. I wanted to reach out before doing that. Thanks!

— Reply to this email directly, view it on GitHub https://github.com/CodeForPhilly/clean-and-green-philly/issues/751#issuecomment-2269749387, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAPKRGQJHOGYJM2NJT33PVDZP7GABAVCNFSM6AAAAABK7ABBGKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRZG42DSMZYG4 . You are receiving this because you were mentioned.Message ID: @.***>

csharpie commented 1 month ago

Hi Team,

I unfortunately hit a bit of roadblock with getting the yml file built for this and I unfortunately don't have a working GitHub workflow yml file to commit. Here's what I've learned so far in hopes that this helps whomever takes this issue on next.

SonarQube is not available in a clouded hosted Community Edition but SonarCloud is and is available at the Free Tier for Open Source Projects! I'd highly recommend reading through this Dev.to article. These are articles and documentation that I found helpful for getting me up to speed on SonarCloud with GitHub Actions: