Skip to content

Comments

Bump workflows actions#974

Merged
barryvdh merged 2 commits intophp-debugbar:masterfrom
erikn69:patch-76
Jan 27, 2026
Merged

Bump workflows actions#974
barryvdh merged 2 commits intophp-debugbar:masterfrom
erikn69:patch-76

Conversation

@erikn69
Copy link
Contributor

@erikn69 erikn69 commented Jan 27, 2026

No description provided.

@erikn69 erikn69 force-pushed the patch-76 branch 4 times, most recently from 56c92d4 to fe0de16 Compare January 27, 2026 16:56
@erikn69
Copy link
Contributor Author

erikn69 commented Jan 27, 2026

Wouldn't build-assets.yml and fix-code-style.yml qualify under race conditions?
I mean, if both have to do autocommit, one does it first and the other can't.
should they join in a single workflow?

https://github.com/stefanzweifel/git-auto-commit-action?tab=readme-ov-file#use-in-forks-from-public-repositories

If you use this Action in combination with a linter/fixer, it's easier if you run the Action on push on your main-branch.

By default, this Action will not run on Pull Requests which have been opened by forks. (This is a limitation by GitHub, not by us.)

should pull_request: part be removed?

push:
branches:
- master
paths:
- 'resources/**'
pull_request:
branches:
- "*"
paths:
- 'resources/**'

@barryvdh
Copy link
Collaborator

Hmm it does run for myself, but not sure if I can exclude others.

@barryvdh barryvdh merged commit 21bbb20 into php-debugbar:master Jan 27, 2026
21 checks passed
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.

2 participants