Skip to content

docs(GitHub-Actions): Differentiate step names#2

Merged
Kurt-von-Laven merged 1 commit intomainfrom
github-actions
Apr 21, 2022
Merged

docs(GitHub-Actions): Differentiate step names#2
Kurt-von-Laven merged 1 commit intomainfrom
github-actions

Conversation

@Kurt-von-Laven
Copy link
Copy Markdown
Contributor

Make the workflow logs easier to read by clarifying the rationale for having two separate "Install and run pre-commit hooks" steps. One handles the case where the workflow is called from within this repository, and the other handles the case where it's called externally.

Make the workflow logs easier to read by clarifying the rationale for
having two separate "Install and run pre-commit hooks" steps. One
handles the case where the workflow is called from within this
repository, and the other handles the case where it's called externally.
@Kurt-von-Laven Kurt-von-Laven requested a review from mwarres April 21, 2022 03:20
@Kurt-von-Laven Kurt-von-Laven self-assigned this Apr 21, 2022
Copy link
Copy Markdown
Contributor

@mwarres mwarres left a comment

Choose a reason for hiding this comment

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

Good clarification!

@Kurt-von-Laven Kurt-von-Laven merged commit 6c1a4aa into main Apr 21, 2022
@Kurt-von-Laven Kurt-von-Laven deleted the github-actions branch April 21, 2022 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants