Skip to content

Commit e5a3934

Browse files
authored
Merge branch 'main' into hiaga/docFixes
2 parents 32e2d40 + 260ed6a commit e5a3934

File tree

247 files changed

+24905
-2694
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

247 files changed

+24905
-2694
lines changed

.github/dependabot.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,9 @@ updates:
3838
eslint:
3939
patterns:
4040
- "*eslint*"
41+
openapi-validator:
42+
patterns:
43+
- "@microsoft.azure/openapi-validator*"
4144
versioning-strategy: increase-if-necessary
4245
- package-ecosystem: "npm"
4346
directories:

.github/workflows/src/summarize-checks/summarize-checks.js

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -363,6 +363,12 @@ export async function summarizeChecksImpl(
363363
) {
364364
core.info(`Handling ${event_name} event for PR #${issue_number} in ${owner}/${repo}.`);
365365

366+
const prUrl = `https://github.com/${owner}/${repo}/pull/${issue_number}`;
367+
core.summary.addRaw("PR: ");
368+
core.summary.addLink(prUrl, prUrl);
369+
core.summary.write();
370+
core.setOutput("summary", process.env.GITHUB_STEP_SUMMARY);
371+
366372
let labelNames = await getExistingLabels(github, owner, repo, issue_number);
367373

368374
/** @type {[CheckRunData[], CheckRunData[], import("./labelling.js").ImpactAssessment | undefined]} */
@@ -441,6 +447,9 @@ export async function summarizeChecksImpl(
441447
core.info(
442448
`Updating comment '${NEXT_STEPS_COMMENT_ID}' on ${owner}/${repo}#${issue_number} with body: ${commentBody}`,
443449
);
450+
core.summary.addRaw(`\n${commentBody}\n\n`);
451+
core.summary.write();
452+
444453
// this will remain commented until we're comfortable with the change.
445454
// await commentOrUpdate(
446455
// { github, context, core },
@@ -457,6 +466,9 @@ export async function summarizeChecksImpl(
457466
core.info(
458467
`Summarize checks has identified that status of "[TEST-IGNORE] Automated merging requirements met" commit status should be updated to: ${JSON.stringify(automatedChecksMet)}.`,
459468
);
469+
core.summary.addHeading("Automated Checks Met", 2);
470+
core.summary.addCodeBlock(JSON.stringify(automatedChecksMet, null, 2));
471+
core.summary.write();
460472
}
461473

462474
/**

.github/workflows/summarize-checks.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,3 +68,13 @@ jobs:
6868
const { default: summarizeChecks } =
6969
await import('${{ github.workspace }}/.github/workflows/src/summarize-checks/summarize-checks.js');
7070
return await summarizeChecks({ github, context, core });
71+
72+
# May be used by downstream workflows
73+
- name: Set job-summary artifact
74+
if: ${{ always() && steps.summarize-checks.outputs.summary }}
75+
uses: actions/upload-artifact@v4
76+
with:
77+
name: job-summary
78+
path: ${{ steps.summarize-checks.outputs.summary }}
79+
# If the file doesn't exist, just don't add the artifact
80+
if-no-files-found: ignore

eng/tools/lint-diff/package.json

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,9 @@
2121
"dependencies": {
2222
"@apidevtools/json-schema-ref-parser": "^14.1.1",
2323
"@azure-tools/specs-shared": "file:../../../.github/shared",
24-
"@microsoft.azure/openapi-validator": "^2.2.4",
24+
"@microsoft.azure/openapi-validator": "2.2.4",
25+
"@microsoft.azure/openapi-validator-core": "1.0.6",
26+
"@microsoft.azure/openapi-validator-rulesets": "2.1.7",
2527
"autorest": "^3.7.2",
2628
"axios": "^1.8.3",
2729
"change-case": "^5.4.4",

0 commit comments

Comments
 (0)