actions/dependency-review-action
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge branch 'juxtin/snapshot-warnings' into juxtin/dr-snaps-pre-launch

Browse Source
This commit is contained in:
Justin Holguín
2023-08-31 16:06:14 +00:00
committed by GitHub
parent 15e91a3980 ee86529290
commit abc80cf6a0
3 changed files with 1 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
dist/index.js generated vendored
View File

@@ -1061,14 +1061,6 @@ function addScannedDependencies(changes) {
}
exports.addScannedDependencies = addScannedDependencies;
function addSnapshotWarnings(warnings) {
// For now, we want to ignore warnings that just complain
// about missing snapshots on the head SHA. This is a product
// decision to avoid presenting warnings to users who simply
// don't use snapshots.
const ignore_regex = new RegExp(/No.*snapshot.*found.*head.*/, 'i');
if (ignore_regex.test(warnings)) {
return;
}
core.summary.addHeading('Snapshot Warnings', 2);
core.summary.addQuote(`${icons.warning}: ${warnings}`);
core.summary.addRaw('Re-running this action after a short time may resolve the issue. See the documentation for more information and troubleshooting advice.');

2
dist/index.js.map generated vendored
View File

File diff suppressed because one or more lines are too long

9
src/summary.ts
View File

@@ -232,15 +232,6 @@ export function addScannedDependencies(changes: Changes): void {
}
export function addSnapshotWarnings(warnings: string): void {
// For now, we want to ignore warnings that just complain
// about missing snapshots on the head SHA. This is a product
// decision to avoid presenting warnings to users who simply
// don't use snapshots.
const ignore_regex = new RegExp(/No.*snapshot.*found.*head.*/, 'i')
if (ignore_regex.test(warnings)) {
return
}
core.summary.addHeading('Snapshot Warnings', 2)
core.summary.addQuote(`${icons.warning}: ${warnings}`)
core.summary.addRaw(