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.

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
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
59 changes: 55 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -77,11 +77,15 @@ Note Flathub's hosted tool only checks the default branch.

To stop Flathub's tool from checking your repo, add `"disable-external-data-checker": true` to `flathub.json` in the default branch.

### Custom workflow
### Custom workflows

Alternatively, you can use own workflow. This can be useful if e.g. wanting to update non-default branches.

Put this yaml file under `.github/workflows`, e.g. put it in `.github/workflows/update.yaml`. Ensure to put the correct path to the manifest in the last line.
There are two workflows examples, with the only difference being which repository is pushed to for changes.
The first workflow has f-e-d-c push changes to a branch in the same repository where the workflow is run.
The second instead pushes to a branch in a fork of the repository.

Put either yaml file under `.github/workflows`, e.g. put it in the file `.github/workflows/update.yaml`. Ensure to put the correct path to the manifest in the last line.

```yaml
name: Check for updates
Expand All @@ -96,7 +100,7 @@ jobs:

strategy:
matrix:
branch: [ master ] # list all branches to check
branch: [ master, beta ] # list all branches to check

steps:
- uses: actions/checkout@v3
Expand All @@ -111,11 +115,58 @@ jobs:
GIT_AUTHOR_EMAIL: 41898282+github-actions[bot]@users.noreply.github.com
GIT_COMMITTER_EMAIL: 41898282+github-actions[bot]@users.noreply.github.com
EMAIL: 41898282+github-actions[bot]@users.noreply.github.com
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # gives permission to push changes to the repository
with:
args: --update --never-fork $PATH_TO_MANIFEST # e.g. com.organization.myapp.json
```

This second workflow has the tool push updates to a fork of your repository. This may be preferred if you do not want the tool to have direct push access to the canonical repository.
You will need to provide a token of a GitHub user to fork the repository and make PRs.
It is recommended to not use your main account as the updater/bot user, since any collaborators of the repository will have access to the GitHub Secret containing the user's token.

In addition to adding this workflow file to your repository, [create a PAT for the user which will fork and create PRs](https://docs.github.com/en/authentication/keeping-your-account-and-data-secure/creating-a-personal-access-token#creating-a-token) that has repo:public_repos access. Note if you have a private repo you will need to provide full repo permission.

Then, [create a GitHub Secret](https://docs.github.com/en/actions/security-guides/encrypted-secrets#creating-encrypted-secrets-for-a-repository) in the repository the workflow is being added to, with the name matching the workflow's GITHUB_TOKEN value. The Secret's value should be the PAT itself.

```yaml
name: Check for updates
on:
schedule: # for scheduling to work this file must be in the default branch
- cron: "0 * * * *" # run every hour
workflow_dispatch: # can be manually dispatched under GitHub's "Actions" tab

jobs:
flatpak-external-data-checker:
runs-on: ubuntu-latest

strategy:
matrix:
branch: [ master, beta ] # list all branches to check

steps:
- uses: actions/checkout@v3
with:
ref: ${{ matrix.branch }}
# we can't push changes to a fork via a shallow update, we need to fetch the full repo
fetch-depth: 0
# by default the local git config will use the workflow provided GITHUB_TOKEN/credentials to push changes
# this will fail to push to a fork since that config only provides access to the repository where the workflow is running
persist-credentials: false

- uses: docker://ghcr.io/flathub/flatpak-external-data-checker:latest
env:
# replace these with the name and email of the user you wish to fork and make PRs
# GitHub's autogenerated noreply user email address can be copied from the page mentioned here: https://docs.github.com/en/account-and-profile/setting-up-and-managing-your-personal-account-on-github/managing-email-preferences/setting-your-commit-email-address#setting-your-commit-email-address-on-github
GIT_AUTHOR_NAME: <Updater GitHub user name>
GIT_COMMITTER_NAME: <Updater GitHub user name>
GIT_AUTHOR_EMAIL: <Updater GitHub user email>
GIT_COMMITTER_EMAIL: <Updater GitHub user email>
EMAIL: <Updater GitHub user email>
GITHUB_TOKEN: ${{ secrets.BOT_USER_SECRET }} # replace this with the name of the GitHub Secret where you store the PAT of the user you wish to fork the repo and create PRs
with:
args: --update --always-fork $PATH_TO_MANIFEST # e.g. com.organization.myapp.json
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't strictly need --always-fork since the script will check whether it has push access to the main repo, and use a fork if not.

But I can see it is good for documentation value.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I did for documentation value especially since the other example uses --never-fork

```

### Automatically submitting PRs

When run with the `--update` flag, this tool can commit any necessary changes
Expand Down