Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update rit.yml to use v2 of RIT #3001

Merged
merged 2 commits into from
Feb 24, 2025
Merged

Update rit.yml to use v2 of RIT #3001

merged 2 commits into from
Feb 24, 2025

Conversation

jeremy-then
Copy link
Contributor

@jeremy-then jeremy-then commented Feb 24, 2025

Description

Motivation and Context

How Has This Been Tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • Tests for the changes have been added (for bug fixes / features)
  • Requires Activation Code (Hard Fork)
  • Other information:

Copy link

github-actions bot commented Feb 24, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails
actions/rsksmart/rootstock-integration-tests 8f98dae546a3da7f1b59f3b6381202208aef40e3 🟢 8.3
Details
CheckScoreReason
Binary-Artifacts🟢 10no binaries found in the repo
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: githubv4.Query: Resource not accessible by integration
CI-Tests🟢 104 out of 4 merged PRs checked by a CI test -- score normalized to 10
CII-Best-Practices⚠️ 2badge detected: InProgress
Code-Review🟢 10all changesets reviewed
Contributors🟢 10project has 3 contributing companies or organizations -- score normalized to 10
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Dependency-Update-Tool🟢 10update tool detected
Fuzzing⚠️ 0project is not fuzzed
License🟢 10license file detected
Maintained🟢 1030 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 10
Packaging🟢 10packaging workflow detected
Pinned-Dependencies🟢 8dependency not pinned by hash detected -- score normalized to 8
SAST🟢 10SAST tool is run on all commits
Security-Policy🟢 10security policy file detected
Signed-Releases⚠️ -1no releases found
Token-Permissions🟢 10GitHub workflow tokens follow principle of least privilege
Vulnerabilities⚠️ 011 existing vulnerabilities detected

Scanned Files

  • .github/workflows/rit.yml

@jeremy-then jeremy-then force-pushed the run-rit-2 branch 2 times, most recently from eb27623 to 8bd03f9 Compare February 24, 2025 16:31
@marcos-iov marcos-iov changed the title Run rit 2 Update rit.yml to use v2 of RIT Feb 24, 2025
Copy link
Collaborator

@josedahlquist josedahlquist left a comment

Choose a reason for hiding this comment

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

I will approve this but I would suggest to try pointing to master instead:

uses: rsksmart/rootstock-integration-tests

@josedahlquist josedahlquist merged commit 32651f8 into master Feb 24, 2025
12 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants