Skip to content

Improve log messages - #730

Open
bluwy wants to merge 1 commit into
mainfrom
improve-log-messages
Open

Improve log messages#730
bluwy wants to merge 1 commit into
mainfrom
improve-log-messages

Conversation

@bluwy

@bluwy bluwy commented Aug 26, 2026

Copy link
Copy Markdown
Member

Did a subjective pass and tried to improve/make more consistent of the log messages

Comment thread src/index.ts
: ""
}`,
);
process.exit(result.exitCode);

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't needed as I changed to a throw above, which core.setFailed will make the exit code 1 by default. But while this might be a different exit code than result.exitCode, it doesn't matter in practice as GitHub Actions only has two states: success and failure, which are checked by 0 and non-0 only.

Comment thread src/run.ts
base: branch,
});
core.info(
core.debug(

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I find this info to bloat the logs in practice, so moved this to debug, which can be viewed when the user re-runs the action in debug mode.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant