Skip to content
This repository was archived by the owner on Sep 26, 2023. It is now read-only.

chore: Add SonarCloud support #1614

Merged
merged 3 commits into from
Jan 28, 2022
Merged

chore: Add SonarCloud support #1614

merged 3 commits into from
Jan 28, 2022

Conversation

meltsufin
Copy link
Member

SonarCloud provides code coverage and static code quality analysis.
https://sonarcloud.io/project/overview?id=googleapis_gax-java

@meltsufin meltsufin requested a review from chanseokoh January 28, 2022 19:24
@meltsufin meltsufin requested review from a team as code owners January 28, 2022 19:24
Copy link
Contributor

@chanseokoh chanseokoh left a comment

Choose a reason for hiding this comment

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

LGTM, pending some comments.

uses: actions/cache@v1
with:
path: ~/.sonar/cache
key: ${{ runner.os }}-sonar
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this correct? It will never be recached once done? In another repo, I see

key: ${{ runner.os }}-sonar-${{ steps.date.outputs.date }}

which looks more reasonable. (However, I don't understand restore-keys, so I may be missing something.)

Copy link
Member Author

Choose a reason for hiding this comment

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

That's the default with the issue of potentially having a very large cache over the long term. For now, I think we can just proceed with the default.

meltsufin and others added 2 commits January 28, 2022 14:38
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@meltsufin meltsufin merged commit db620af into main Jan 28, 2022
@meltsufin meltsufin deleted the sonar branch January 28, 2022 20:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants