actions on push branches single value not triggering #14161

Closed
opened 2025-11-02 11:04:52 -06:00 by GiteaMirror · 3 comments
Owner

Originally created by @aceArt-GmbH on GitHub (Feb 20, 2025).

Description

Up until gitea 1.22 following action config was valid and triggert on a push to master.

name: Unit tests
on:
    push:
        branches: master

After updating to 1.23 this no longer triggers a build.

Workaround:

name: Unit tests
on:
    push:
        branches:
            - master

The "workaround" is simple and may be the (only) correct thing.
But the changelog did not mention this change, so I suspect it being a regression.
I am fine with either, but it should be fixed OR mentioned as breaking change 🙃

Gitea Version

1.23.4

Can you reproduce the bug on the Gitea demo site?

No

Log Gist

No response

Screenshots

No response

Git Version

2.47.2

Operating System

alpine lxc

How are you running Gitea?

docker compose

Database

MySQL/MariaDB

Originally created by @aceArt-GmbH on GitHub (Feb 20, 2025). ### Description Up until gitea 1.22 following action config was valid and triggert on a push to master. ```yml name: Unit tests on: push: branches: master ``` After updating to 1.23 this no longer triggers a build. Workaround: ```yml name: Unit tests on: push: branches: - master ``` The "workaround" is simple and may be the (only) correct thing. But the changelog did not mention this change, so I suspect it being a regression. I am fine with either, but it should be fixed OR mentioned as breaking change 🙃 ### Gitea Version 1.23.4 ### Can you reproduce the bug on the Gitea demo site? No ### Log Gist _No response_ ### Screenshots _No response_ ### Git Version 2.47.2 ### Operating System alpine lxc ### How are you running Gitea? docker compose ### Database MySQL/MariaDB
GiteaMirror added the topic/gitea-actionstype/bug labels 2025-11-02 11:04:52 -06:00
Author
Owner

@gsvd commented on GitHub (Feb 21, 2025):

Hey, I've submitted a bug fix in this PR.
This issue is related to the gitea/act dependency.

@gsvd commented on GitHub (Feb 21, 2025): Hey, I've submitted a bug fix in [this PR](https://gitea.com/gitea/act/pulls/129). This issue is related to the [gitea/act](https://gitea.com/gitea/act) dependency.
Author
Owner

@aceArt-GmbH commented on GitHub (Feb 26, 2025):

Should I close this as https://gitea.com/gitea/act/pulls/129 is merged?
Or does gitea/act still need updating here?

@aceArt-GmbH commented on GitHub (Feb 26, 2025): Should I close this as https://gitea.com/gitea/act/pulls/129 is merged? Or does gitea/act still need updating here?
Author
Owner

@lunny commented on GitHub (Feb 26, 2025):

Should I close this as https://gitea.com/gitea/act/pulls/129 is merged? Or does gitea/act still need updating here?

Both act_runner and gitea need to be updated before we can proceed with closing this issue.

@lunny commented on GitHub (Feb 26, 2025): > Should I close this as https://gitea.com/gitea/act/pulls/129 is merged? Or does gitea/act still need updating here? Both act_runner and gitea need to be updated before we can proceed with closing this issue.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#14161