> Sounds good.
> 
>     * For the next time, a description to describe what is the PR about and 
> more importantly why, is generally a good idea.

Understood!

> 
>     * We could add `*.py` to the patterns and `python` to the matrix to also 
> cover the Python helper scripts in the repository.

Done.

> 
>     * Also, could you strip the commented code from the workflow definition 
> and also the extensive description comments? I think they are not very useful.

Done.

>Adding   workflow_dispatch: might be a good idea to manually re-run the 
>workflow.

Done. Some examples of this workflow include a cron schedule as well. Would you 
like that added, and if so, how often should it be scheduled?



-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3769#issuecomment-1951462845
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3769/c1951462...@github.com>

Reply via email to