Compare commits

...

5 Commits

Author SHA1 Message Date
Scott Crooks
7efdf018c8
Merge 4c5aaae1a5 into f3ceefdc7e 2026-04-13 16:20:29 +00:00
Sascha Bratton
f3ceefdc7e
fix: use rev-parse instead of branch --show-current for older git compat (#303) 2026-04-10 17:44:48 -04:00
Lucas Servén Marín
61f87a10cd
chore: fix GitHub spelling in logs (#278)
* chore: fix GitHub spelling in logs

* fix: rebuild dist

---------

Co-authored-by: Sascha Bratton <sascha@queen.one>
2026-03-25 12:27:15 -04:00
Scott Crooks
4c5aaae1a5
Merge branch 'master' into feat-add-any-all 2025-11-27 10:27:10 +01:00
Scott Crooks
7b47280cac
feat: adding any and all boolean outputs
This adds two boolean outputs `any` and `all` so that someone can track whether (1) any filter has at least one changed file, or (2) all filters have at least one changed file

Closes #266

Signed-off-by: Scott Crooks <scott.crooks@gmail.com>
2025-04-11 22:06:19 +02:00
4 changed files with 24 additions and 8 deletions

View File

@ -50,6 +50,10 @@ inputs:
required: false
default: 'some'
outputs:
all_changed:
description: Boolean value indicating if all of the filters contained at least one changed file
any_changed:
description: Boolean value indicating if any of the filters contained at least one changed file
changes:
description: JSON array with names of all filters matching any of changed files
runs:

14
dist/index.js vendored
View File

@ -363,8 +363,8 @@ exports.listAllFilesAsAdded = listAllFilesAsAdded;
async function getCurrentRef() {
core.startGroup(`Get current git ref`);
try {
const branch = (await (0, exec_1.getExecOutput)('git', ['branch', '--show-current'])).stdout.trim();
if (branch) {
const branch = (await (0, exec_1.getExecOutput)('git', ['rev-parse', '--abbrev-ref', 'HEAD'])).stdout.trim();
if (branch && branch !== 'HEAD') {
return branch;
}
const describe = await (0, exec_1.getExecOutput)('git', ['describe', '--tags', '--exact-match'], { ignoreReturnCode: true });
@ -640,7 +640,7 @@ async function getChangedFiles(token, base, ref, initialFetchDepth) {
// At the same time we don't want to fetch any code from forked repository
throw new Error(`'token' input parameter is required if action is triggered by 'pull_request_target' event`);
}
core.info('Github token is not available - changes will be detected using git diff');
core.info('GitHub token is not available - changes will be detected using git diff');
const baseSha = (_a = github.context.payload.pull_request) === null || _a === void 0 ? void 0 : _a.base.sha;
const defaultBranch = (_b = github.context.payload.repository) === null || _b === void 0 ? void 0 : _b.default_branch;
const currentRef = await git.getCurrentRef();
@ -710,7 +710,7 @@ async function getChangedFilesFromGit(base, head, initialFetchDepth) {
}
// Uses github REST api to get list of files changed in PR
async function getChangedFilesFromApi(token, pullRequest) {
core.startGroup(`Fetching list of changed files for PR#${pullRequest.number} from Github API`);
core.startGroup(`Fetching list of changed files for PR#${pullRequest.number} from GitHub API`);
try {
const client = github.getOctokit(token);
const per_page = 100;
@ -761,11 +761,14 @@ async function getChangedFilesFromApi(token, pullRequest) {
function exportResults(results, format) {
core.info('Results:');
const changes = [];
let anyChanged = false;
let allChanged = true;
for (const [key, files] of Object.entries(results)) {
const value = files.length > 0;
core.startGroup(`Filter ${key} = ${value}`);
if (files.length > 0) {
changes.push(key);
anyChanged = true;
core.info('Matching files:');
for (const file of files) {
core.info(`${file.filename} [${file.status}]`);
@ -773,6 +776,7 @@ function exportResults(results, format) {
}
else {
core.info('Matching files: none');
allChanged = false;
}
core.setOutput(key, value);
core.setOutput(`${key}_count`, files.length);
@ -780,6 +784,8 @@ function exportResults(results, format) {
const filesValue = serializeExport(files, format);
core.setOutput(`${key}_files`, filesValue);
}
core.setOutput('all_changed', allChanged);
core.setOutput('any_changed', anyChanged);
core.endGroup();
}
if (results['changes'] === undefined) {

View File

@ -166,8 +166,8 @@ export async function listAllFilesAsAdded(): Promise<File[]> {
export async function getCurrentRef(): Promise<string> {
core.startGroup(`Get current git ref`)
try {
const branch = (await getExecOutput('git', ['branch', '--show-current'])).stdout.trim()
if (branch) {
const branch = (await getExecOutput('git', ['rev-parse', '--abbrev-ref', 'HEAD'])).stdout.trim()
if (branch && branch !== 'HEAD') {
return branch
}

View File

@ -107,7 +107,7 @@ async function getChangedFiles(token: string, base: string, ref: string, initial
// At the same time we don't want to fetch any code from forked repository
throw new Error(`'token' input parameter is required if action is triggered by 'pull_request_target' event`)
}
core.info('Github token is not available - changes will be detected using git diff')
core.info('GitHub token is not available - changes will be detected using git diff')
const baseSha = github.context.payload.pull_request?.base.sha
const defaultBranch = github.context.payload.repository?.default_branch
const currentRef = await git.getCurrentRef()
@ -194,7 +194,7 @@ async function getChangedFilesFromGit(base: string, head: string, initialFetchDe
// Uses github REST api to get list of files changed in PR
async function getChangedFilesFromApi(token: string, pullRequest: PullRequest): Promise<File[]> {
core.startGroup(`Fetching list of changed files for PR#${pullRequest.number} from Github API`)
core.startGroup(`Fetching list of changed files for PR#${pullRequest.number} from GitHub API`)
try {
const client = github.getOctokit(token)
const per_page = 100
@ -249,17 +249,21 @@ async function getChangedFilesFromApi(token: string, pullRequest: PullRequest):
function exportResults(results: FilterResults, format: ExportFormat): void {
core.info('Results:')
const changes = []
let anyChanged = false
let allChanged = true
for (const [key, files] of Object.entries(results)) {
const value = files.length > 0
core.startGroup(`Filter ${key} = ${value}`)
if (files.length > 0) {
changes.push(key)
anyChanged = true
core.info('Matching files:')
for (const file of files) {
core.info(`${file.filename} [${file.status}]`)
}
} else {
core.info('Matching files: none')
allChanged = false
}
core.setOutput(key, value)
@ -268,6 +272,8 @@ function exportResults(results: FilterResults, format: ExportFormat): void {
const filesValue = serializeExport(files, format)
core.setOutput(`${key}_files`, filesValue)
}
core.setOutput('all_changed', allChanged)
core.setOutput('any_changed', anyChanged)
core.endGroup()
}