Skip to content

Conversation

@mineme0110
Copy link
Contributor

@mineme0110 mineme0110 commented Dec 5, 2025

Description

SAST tool Needed

still need to:

  1. Set up SonarCloud (free for open source):
    - Visit https://sonarcloud.io
    - Sign in with GitHub
    - Import hyperledger-identus/cloud-agent repository
  2. Add GitHub Secret:
    - Generate token in SonarCloud: Project → Administration → Security
    - Add to GitHub: Settings → Secrets and variables → Actions → New secret
    - Name: SONAR_TOKEN
    - Value:

Checklist

  • My PR follows the contribution guidelines of this project
  • My PR is free of third-party dependencies that don't comply with the Allowlist
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked the PR title to follow the conventional commit specification

Signed-off-by: mineme0110 <shailesh.patil@iohk.io>
@mineme0110 mineme0110 requested a review from a team as a code owner December 5, 2025 14:16
@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 2025

Integration Test Results

 24 files  ±0   24 suites  ±0   3s ⏱️ ±0s
 61 tests ±0   61 ✅ ±0  0 💤 ±0  0 ❌ ±0 
113 runs  ±0  113 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 4040b08. ± Comparison against base commit adc81b1.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 2025

Unit Test Results

106 files  ±0  106 suites  ±0   19m 0s ⏱️ -46s
899 tests ±0  891 ✅ ±0  8 💤 ±0  0 ❌ ±0 
906 runs  ±0  898 ✅ ±0  8 💤 ±0  0 ❌ ±0 

Results for commit 4040b08. ± Comparison against base commit adc81b1.

♻️ This comment has been updated with latest results.

@coveralls
Copy link

coveralls commented Dec 5, 2025

Coverage Status

coverage: 47.662%. remained the same
when pulling 4040b08 on add-sonarcube-sast-tool
into adc81b1 on main.

Signed-off-by: mineme0110 <shailesh.patil@iohk.io>
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.

3 participants