Skip to content

Commit 6a09333

Browse files
committed
Address custom skill review feedback
1 parent e940fab commit 6a09333

7 files changed

Lines changed: 12 additions & 7 deletions

.x/skills/find_aaz_fork_prs_ready_for_promotion.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,5 @@
44
def find_aaz_fork_prs_ready_for_promotion():
55
"""Find completed AAZ fork pull requests ready for promotion."""
66
return find_generation_source_fork_prs_ready_for_promotion(
7-
repository=None,
7+
repository="Azure/azure-cli",
88
)

.x/skills/find_promoted_aaz_source_pr.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,6 @@
44
def find_promoted_aaz_source_pr(issue_number):
55
"""Find the promoted AAZ source pull request for an Agent issue."""
66
return find_promoted_generation_source_pr(
7-
repository=None,
7+
repository="Azure/azure-cli",
88
issue_number=issue_number,
99
)

.x/skills/get_pr_regression_coverage_summary.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44
def get_pr_regression_coverage_summary(pr_number):
55
"""Find changed command modules without focused tests or recordings."""
66
changes = get_pr_file_changes(
7-
owner=None,
8-
repo=None,
7+
owner="Azure",
8+
repo="azure-cli",
99
pr_number=pr_number,
1010
)
1111
files = [

.x/skills/promote_aaz_fork_pr.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
def promote_aaz_fork_pr(fork_pr_number, title, body):
55
"""Promote one validated AAZ fork pull request."""
66
return promote_generation_source_fork_pr(
7-
repository=None,
7+
repository="Azure/azure-cli",
88
fork_pr_number=fork_pr_number,
99
title=title,
1010
body=body,

.x/skills/start_aaz_source_task.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
def start_aaz_source_task(issue_number, downstream_pr_url, changed_files, prompt_context):
55
"""Start the configured AAZ source task for an Azure CLI pull request."""
66
return start_generation_source_task(
7-
repository=None,
7+
repository="Azure/azure-cli",
88
issue_number=issue_number,
99
downstream_pr_url=downstream_pr_url,
1010
changed_files=changed_files,

.x/skills/start_extension_tracker_task.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ def start_extension_tracker_task(issue_number, view, target, prompt, command, su
1212
"start_extension_tracker_task requires a CLI Extensions target"
1313
)
1414
return start_repository_handoff_task(
15-
repository=None,
15+
repository="Azure/azure-cli",
1616
issue_number=issue_number,
1717
view=view,
1818
target=target,

.x/tester.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,11 @@ the PR title/body, and `pr_files` set to those filenames. Use
1515
module or test path; repository custom skills own both decisions and the
1616
workflow validates them against the current PR.
1717

18+
If no test path is selected, call the dispatcher with the empty list so it
19+
records a neutral skip for the current revision. If tests are selected but
20+
target inference does not return a named `module` or `extension`, stop with a
21+
pending result and do not dispatch.
22+
1823
Before dispatch, reuse any queued, in-progress, or completed run for the same
1924
head SHA. A new dispatch counts as one action; a reused run is a read. Call
2025
`get_workflow_run` once. If it is not complete, return pending and let a later

0 commit comments

Comments
 (0)