Skip to content
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

task(ci): add sonar #77

Merged
merged 1 commit into from
Nov 9, 2023
Merged

task(ci): add sonar #77

merged 1 commit into from
Nov 9, 2023

Conversation

afflom
Copy link
Contributor

@afflom afflom commented Nov 3, 2023

@afflom afflom force-pushed the PSCE-262 branch 13 times, most recently from 9531399 to 30fa3a9 Compare November 6, 2023 23:19
@afflom afflom marked this pull request as ready for review November 6, 2023 23:24
@afflom afflom requested a review from jpower432 November 6, 2023 23:24
Copy link
Member

@jpower432 jpower432 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. A couple questions/comments.

pull_request:
branches:
- main
types: [opened, synchronize, reopened]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this line is redundant. These activity types are what are set by default. https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request.

-Dsonar.organization=rh-psce
- name: SonarQube Quality Gate check
uses: sonarsource/sonarqube-quality-gate-action@master
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we pin to a commit instead of the master branch?

name: coverage
- name: SonarCloud Scan
uses: SonarSource/sonarcloud-github-action@master
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we pin to a commit instead of the master branch?

Copy link
Member

@jpower432 jpower432 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Do you have test branch showing this running successfully? The workflow ended up being skipped in this instance because there are no python changes.

@afflom
Copy link
Contributor Author

afflom commented Nov 9, 2023

LGTM. Do you have test branch showing this running successfully? The workflow ended up being skipped in this instance because there are no python changes.

I found a phrase with a period that I removed to verify functionality. PTAL.

@jpower432 jpower432 merged commit 727acf8 into main Nov 9, 2023
14 checks passed
@jpower432 jpower432 deleted the PSCE-262 branch December 7, 2023 00:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants