Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve custom workflow documentation and add an example #319

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Improve custom workflow documentation and add an example #319

wants to merge 3 commits into from

Commits on Aug 5, 2022

  1. readme: Add another branch to example workflow 

    This makes it more obvious what the syntax is to add them
Multiple branches is likely a common usecase for this workflow on flathub
    @vchernin
    vchernin committed Aug 5, 2022
    Configuration menu
    Copy the full SHA
    f3a83eb View commit details
    Browse the repository at this point in the history

  2. readme: Add an example workflow that pushes to a fork 

    This requires some non-obvious configuration to work correctly, so this seems suitable here.
Originally I had thought to keep only one actual copy pastable workflow, but since some options like persist-credentials: false strictly only belong in the fork repository workflow, I thought it was more helpful to keep 2 workflows so each use case can copy paste easily.
    @vchernin
    vchernin committed Aug 5, 2022
    Configuration menu
    Copy the full SHA
    538195c View commit details
    Browse the repository at this point in the history

  3. fixup! readme: Add an example workflow that pushes to a fork This req… 

    …uires some non-obvious configuration to work correctly, so this seems suitable here. Originally I had thought to keep only one actual copy pastable workflow, but since some options like persist-credentials: false strictly only belong in the fork repository workflow, I thought it was more helpful to keep 2 workflows so each use case can copy paste easily.
    @vchernin
    vchernin committed Aug 5, 2022
    Configuration menu
    Copy the full SHA
    2a889f4 View commit details
    Browse the repository at this point in the history