Skip to content

Commit

Permalink
fix(STONEINTG-1091): fix typo
Browse files Browse the repository at this point in the history
Signed-off-by: Hongwei Liu <[email protected]>
  • Loading branch information
hongweiliu17 committed Nov 26, 2024
1 parent 2af6142 commit 2893b4e
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion docs/statusreport-controller.md
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ flowchart TD
does_commitStatus_exist{Does commitStatus exist <br>on github already?}
create_new_commitStatus_on_gh(Create new commitStatus on github<br>if PR is not from forked repo)
does_comment_exist(Does a comment exist for snapshot and scenario?)
update_existing_comment(Update the existing comment for <br>snapshot and scenario</br><br>if PR is not from forked repo)
update_existing_comment(Update the existing comment for <br>snapshot and scenario</br>)
create_new_comment(Create a new comment for <br>snapshot and scenario</br>)
collect_commit_info_gl(Collect commit projectID, repo-url and SHA from Snapshot)
Expand Down
2 changes: 1 addition & 1 deletion status/reporter_github.go
Original file line number Diff line number Diff line change
Expand Up @@ -456,7 +456,7 @@ func (csu *CommitStatusUpdater) UpdateStatus(ctx context.Context, report TestRep
"snapshot.NameSpace", csu.snapshot.Namespace, "snapshot.Name", csu.snapshot.Name, "scenarioName", report.ScenarioName)
}
} else {
csu.logger.Info("Won't create/update commitStatus since there is access limimation for different source and target Repo Owner",
csu.logger.Info("Won't create/update commitStatus since there is access limitation for different source and target Repo Owner",
"snapshot.NameSpace", csu.snapshot.Namespace, "snapshot.Name", csu.snapshot.Name, "sourceRepoOwner", sourceRepoOwner, "targetRepoOwner", csu.owner)
}
// Create a comment when integration test is neither pending nor inprogress since comment for pending/inprogress is less meaningful and there is commitStatus for all statuses
Expand Down
2 changes: 1 addition & 1 deletion status/reporter_github_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -607,7 +607,7 @@ var _ = Describe("GitHubReporter", func() {
Summary: "Integration test for snapshot snapshot-sample and scenario scenario2 is pending",
}
Expect(reporter.ReportStatus(context.TODO(), testReport)).To(Succeed())
expectedLogEntry := "Won't create/update commitStatus since there is access limimation for different source and target Repo Owner"
expectedLogEntry := "Won't create/update commitStatus since there is access limitation for different source and target Repo Owner"
Expect(buf.String()).Should(ContainSubstring(expectedLogEntry))
})
})
Expand Down

0 comments on commit 2893b4e

Please sign in to comment.