Skip to content

Commit 720dbf9

Browse files
committed
Minor docs and logging updates
1 parent d67d85a commit 720dbf9

2 files changed

Lines changed: 5 additions & 7 deletions

File tree

src/store/workflowRun.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,13 +75,12 @@ export class WorkflowRun extends WorkflowRunBase {
7575
}
7676

7777
override async fetchJobs(): Promise<WorkflowJob[]> {
78-
logDebug("Getting workflow jobs");
78+
logDebug(`Fetching jobs for workflow run ${this._run.display_title} #${this._run.run_number} (${this._run.id})`);
7979

8080
let jobs: model.WorkflowJob[] = [];
8181

8282
try {
8383
jobs = await this._gitHubRepoContext.client.paginate(
84-
// @ts-expect-error FIXME: Newer Typescript catches a problem that previous didn't. This will be fixed in Octokit bump.
8584
this._gitHubRepoContext.client.actions.listJobsForWorkflowRun,
8685
{
8786
owner: this._gitHubRepoContext.owner,
@@ -153,7 +152,6 @@ export class WorkflowRunAttempt extends WorkflowRunBase {
153152

154153
try {
155154
jobs = await this._gitHubRepoContext.client.paginate(
156-
// @ts-expect-error FIXME: Newer Typescript catches a problem that previous didn't. This will be fixed in Octokit bump.
157155
this._gitHubRepoContext.client.actions.listJobsForWorkflowRunAttempt,
158156
{
159157
owner: this._gitHubRepoContext.owner,

src/treeViews/currentBranch.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import {getCurrentBranch, getGitHubContext, GitHubRepoContext} from "../git/repo
55
import {CurrentBranchRepoNode} from "./current-branch/currentBranchRepoNode";
66

77
import {NoRunForBranchNode} from "./current-branch/noRunForBranchNode";
8-
import {log, logDebug, logTrace} from "../log";
8+
import {logDebug, logTrace, logWarn} from "../log";
99
import {RunStore} from "../store/store";
1010
import {AttemptNode} from "./shared/attemptNode";
1111
import {GitHubAPIUnreachableNode} from "./shared/gitHubApiUnreachableNode";
@@ -67,7 +67,7 @@ export class CurrentBranchTreeProvider
6767
const repoContext = gitHubContext.repos[0];
6868
const currentBranch = getCurrentBranch(repoContext.repositoryState);
6969
if (!currentBranch) {
70-
log(`Could not find current branch for ${repoContext.name}`);
70+
logWarn(`Could not find current branch for ${repoContext.name}`);
7171
return [];
7272
}
7373

@@ -79,7 +79,7 @@ export class CurrentBranchTreeProvider
7979
.map((repoContext): CurrentBranchRepoNode | undefined => {
8080
const currentBranch = getCurrentBranch(repoContext.repositoryState);
8181
if (!currentBranch) {
82-
log(`Could not find current branch for ${repoContext.name}`);
82+
logWarn(`Could not find current branch for ${repoContext.name}`);
8383
return undefined;
8484
}
8585

@@ -103,7 +103,7 @@ export class CurrentBranchTreeProvider
103103
}
104104

105105
private async getRuns(gitHubRepoContext: GitHubRepoContext, currentBranchName: string): Promise<WorkflowRunNode[]> {
106-
logDebug("Getting workflow runs for branch");
106+
logDebug(`Getting current branch (${currentBranchName}) runs in repo ${gitHubRepoContext.name}`);
107107

108108
const result = await gitHubRepoContext.client.actions.listWorkflowRunsForRepo({
109109
owner: gitHubRepoContext.owner,

0 commit comments

Comments
 (0)