Apply suggestions from code review

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
This commit is contained in:
Aaron Powell
2026-06-16 11:54:40 +10:00
committed by GitHub
parent de5b159927
commit 83f7cc37a8
2 changed files with 11 additions and 3 deletions
@@ -145,7 +145,14 @@ jobs:
summary: 'No changed external plugin entries were detected in this PR.', summary: 'No changed external plugin entries were detected in this PR.',
}; };
if (shouldRun) { if ('${{ needs.detect-changed-plugins.result }}' === 'failure' || '${{ needs.detect-changed-plugins.result }}' === 'cancelled') {
qualityResult = {
overall_status: 'infra_error',
failure_class: 'infra',
checked_plugins: [],
summary: 'External plugin PR change detection failed unexpectedly. Re-run this workflow.',
};
} else if (shouldRun) {
if (qualityJobResult === 'failure' || qualityJobResult === 'cancelled') { if (qualityJobResult === 'failure' || qualityJobResult === 'cancelled') {
qualityResult = { qualityResult = {
overall_status: 'infra_error', overall_status: 'infra_error',
@@ -167,7 +174,7 @@ jobs:
const stateLabel = qualityResult.failure_class === 'submitter_fixes' const stateLabel = qualityResult.failure_class === 'submitter_fixes'
? 'requires-submitter-fixes' ? 'requires-submitter-fixes'
: qualityResult.overall_status === 'pass' : qualityResult.overall_status === 'pass' || !shouldRun
? 'ready-for-review' ? 'ready-for-review'
: 'awaiting-review'; : 'awaiting-review';
+2 -1
View File
@@ -34,7 +34,8 @@ export function buildSourceTreeUrl(plugin) {
return `https://github.com/${sourceRepo}/tree/${encodedLocator}`; return `https://github.com/${sourceRepo}/tree/${encodedLocator}`;
} }
return `https://github.com/${sourceRepo}/tree/${encodedLocator}/${normalizedPath}`; const encodedPath = encodePathLikeValue(normalizedPath);
return `https://github.com/${sourceRepo}/tree/${encodedLocator}/${encodedPath}`;
} }
function aggregateResultStatus(pluginResults) { function aggregateResultStatus(pluginResults) {