Skip to content

Commit

Permalink
Revert "workflows: fix code style workflows user name (#410)"
Browse files Browse the repository at this point in the history
Because this will assign potentially a lot of additional code changes to only the pipeline triggerer.
Example: Multiple deves are working on the same PR. One of them merges it in the end (or better
a maintainer of us) and "commits" all the changes. This will cause in Git blame to only show him up.
Also, filtering based on the actions email will not work anymore.
Furthermore, people signing commits (like me) will cause a lot of unverified commits to pop up.

This reverts commit 22e3bfa.
  • Loading branch information
simonmicro committed Oct 9, 2024
1 parent 78a0515 commit f219502
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/astyle.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,8 @@ jobs:
- name: Push changes
if: steps.git-check.outputs.modified == 'true'
run: |
git config --global user.email $(jq -r '.pusher.email' $GITHUB_EVENT_PATH)
git config --global user.name $(jq -r '.pusher.name' $GITHUB_EVENT_PATH)
git config --global user.email 'action@github.com'
git config --global user.name 'GitHub Action'
git commit -am 'Applied formatting'
- name: Push changes
uses: ad-m/github-push-action@master
Expand Down

0 comments on commit f219502

Please sign in to comment.