Skip to content

Add command to open remote query on github#1184

Merged
aeisenberg merged 1 commit intomainfrom
aeisenberg/open-remote
Mar 4, 2022
Merged

Add command to open remote query on github#1184
aeisenberg merged 1 commit intomainfrom
aeisenberg/open-remote

Conversation

@aeisenberg
Copy link
Copy Markdown
Contributor

Command is available for remote queries that are in progress or
completed.

Checklist

  • [n/a] CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

@aeisenberg aeisenberg requested a review from a team as a code owner March 3, 2022 18:59
@aeisenberg aeisenberg requested a review from a team March 3, 2022 18:59
Copy link
Copy Markdown
Contributor

@charisk charisk left a comment

Choose a reason for hiding this comment

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

I've not tested this but it LGTM. Just a console.log to remove.

}

const { actionsWorkflowRunId: workflowRunId, controllerRepository: { owner, name } } = finalSingleItem.remoteQuery;
console.log(workflowRunId, owner, name);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Oops

Command is available for remote queries that are in progress or
completed.
@aeisenberg aeisenberg force-pushed the aeisenberg/open-remote branch from 26dc188 to d5549f2 Compare March 4, 2022 18:37
@aeisenberg aeisenberg enabled auto-merge March 4, 2022 18:37
@aeisenberg aeisenberg merged commit 04b8681 into main Mar 4, 2022
@aeisenberg aeisenberg deleted the aeisenberg/open-remote branch March 4, 2022 18:46
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.

2 participants