Fix github ref in main.yml

[kaitlinnewson]

According to the docs for github.head_ref:

The head_ref or source branch of the pull request in a workflow run. This property is only available when the event that triggers a workflow run is either pull_request or pull_request_target.

[kaitlinnewson]

@withanage Can you review this change to the GitHub actions? I noticed that some of the jobs aren't being triggered (e.g. to build the website).

[withanage]

@kaitlinnewson i reviewed the changes. the change is ok to be merged.

[kaitlinnewson]

@withanage are you able to merge this as well? I don't have permissions on this repo.

[kaitlinnewson]

Hi @asmecher, can you give this a second review? I'm trying to resolve the issues with the plugin compatibility site not rebuilding. Thanks!

[asmecher]

Looks fine to me -- though the only material change I think is the branch name, right? In any case, Dulip's OK is what matters, and a skim looks fine from my side. Merged!

[withanage]

Hi @kaitlinnewson and @asmecher

Sorry @kaitlinnewson I missed the merge request.

I have seen that there is an error in the plugin-gallery in the site generation. I added the github user to pkp-plugin-cli (similar to pkp-gihtub-cations) and it should then update the other repo correctly.

https://github.com/pkp/pkp-plugin-cli/pull/10/files

[kaitlinnewson]

@asmecher Since the GitHub Actions are pulling the package from npm, we'll need another release for the npm package to confirm the fix.

[asmecher]

@kaitlinnewson and @withanage, I released a new pkp-plugin-cli package and re-ran the Plugin Gallery build; now it errors out with permission denied:

I suppose the script is trying to push with the github credentials you specified, Dulip, which won't work.

From the Access page in the plugin-compatibility repo, we have a single-purpose github account for that purpose:

I suppose there's probably a github Personal Access Token associated with the push. I can regenerate that if needed, but I suspect there are more modern ways to accomplish this since this was set up.