Skip to content

Commit

Permalink
Merge pull request #61 from tun0/fix-grouping
Browse files Browse the repository at this point in the history
Fix grouping of changes
  • Loading branch information
dorny committed Jan 26, 2021
2 parents 4e7fcc3 + b1a097e commit 363576b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion dist/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -4810,6 +4810,7 @@ function exportResults(results, format) {
const filesValue = serializeExport(files, format);
core.setOutput(`${key}_files`, filesValue);
}
core.endGroup();
}
if (results['changes'] === undefined) {
const changesJson = JSON.stringify(changes);
Expand All @@ -4819,7 +4820,6 @@ function exportResults(results, format) {
else {
core.info('Cannot set changes output variable - name already used by filter output');
}
core.endGroup();
}
function serializeExport(files, format) {
const fileNames = files.map(file => file.filename);
Expand Down
2 changes: 1 addition & 1 deletion src/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -194,6 +194,7 @@ function exportResults(results: FilterResults, format: ExportFormat): void {
const filesValue = serializeExport(files, format)
core.setOutput(`${key}_files`, filesValue)
}
core.endGroup()
}

if (results['changes'] === undefined) {
Expand All @@ -203,7 +204,6 @@ function exportResults(results: FilterResults, format: ExportFormat): void {
} else {
core.info('Cannot set changes output variable - name already used by filter output')
}
core.endGroup()
}

function serializeExport(files: File[], format: ExportFormat): string {
Expand Down

0 comments on commit 363576b

Please sign in to comment.