Skip to content

ci(infer): fix fb-infer-report for prs on forks#372

Open
mojomex wants to merge 1 commit intotier4:mainfrom
mojomex:ci/fix-fb-infer-for-forks
Open

ci(infer): fix fb-infer-report for prs on forks#372
mojomex wants to merge 1 commit intotier4:mainfrom
mojomex:ci/fix-fb-infer-for-forks

Conversation

@mojomex
Copy link
Collaborator

@mojomex mojomex commented Oct 29, 2025

PR Type

  • Bug Fix

Related Links

Description

The fb-infer-report step was using a way to check out repositories that did not work on forks. THis PR fixes this.

Review Procedure

Remarks

Pre-Review Checklist for the PR Author

PR Author should check the checkboxes below when creating the PR.

  • Assign PR to reviewer

Checklist for the PR Reviewer

Reviewers should check the checkboxes below before approval.

  • Commits are properly organized and messages are according to the guideline
  • (Optional) Unit tests have been written for new behavior
  • PR title describes the changes

Post-Review Checklist for the PR Author

PR Author should check the checkboxes below before merging.

  • All open points are addressed and tracked via issues or tickets

CI Checks

  • Build and test for PR: Required to pass before the merge.

Signed-off-by: Max SCHMELLER <max.schmeller@tier4.jp>
@mojomex mojomex added the run:fb-infer Run advanced static analysis for memory and thread safety. Takes up to 30min per run. label Oct 29, 2025
@mojomex mojomex self-assigned this Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run:fb-infer Run advanced static analysis for memory and thread safety. Takes up to 30min per run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant