ubiquity / ts-template

A template repository for all @ubiquity projects.
2 stars 24 forks source link

ci: update empty-string-checker.ts #84

Closed rndquu closed 19 hours ago

rndquu commented 19 hours ago

Check this failing empty string check CI run which fails with the following error:

Error: An error occurred: Resource not accessible by integration - https://docs.github.com/rest/checks/runs#create-a-check-run

The pull_request trigger runs in the context of a forked repository. In order to use the checks.create() API the workflow must use the checks: write permission. But forked instances can only have the read permission.

This PR removes the checks.create() call so that forked instances were not failing. This way the CI is still failing and shows meaningful warnings.

QA: https://github.com/rndquu/ts-template/actions/runs/11953756844/job/33322322490?pr=6

ubiquity-os-deployer[bot] commented 19 hours ago
094401e
rndquu commented 19 hours ago

This is a big downgrade from showing in the files changed view. Surely there is another workaround that preserves this capability?

https://github.com/ubiquity/ts-template/issues/85