-
Notifications
You must be signed in to change notification settings - Fork 21
Common Errors: Restyle PR not created
The following are scenarios in which Restyled may not create a PR:
If we encounter an error and are unable to open the PR. If this is the case, you should see:
- A Red Restyled PR status
- Details about some error in the Job log
If nothing needed fixing, no PR will be created. If this is the case, you should see
- A Green Restyled PR status
- "Restyling produced no differences" in the Job log
If your restyled.yaml
contains pull_requests:false
, a PR will not be created.
https://github.com/restyled-io/restyled.io/wiki/Configuring-Restyled#pull-requests
Restyled cannot create a sibling PR in the fork repository since it may not be installed there.
To work around this, Restyled used to create a PR in the base repository containing the external contributions and the style fixes together. This was a security problem. Fork PRs come with guard rails to safely handle external contributions (e.g. hiding secrets, requiring approval) because such contributions can update CI configuration to (e.g.) exfiltrate data or otherwise compromise the system. Non-fork PRs have none of these guardrails -- as they're expected to be from contributors with write access to the repository. Restyled was effectively using its write access to make external contributions in a non-fork PR. Not great.
Therefore, Restyled will not create PRs for this scenario any more.
Instead, the Job log will direct your contributor to apply the fixes locally
Removed