documentation icon indicating copy to clipboard operation
documentation copied to clipboard

Allow merging of external PRs

Open lorensr opened this issue 3 years ago • 0 comments

Issue

This PR:

https://github.com/temporalio/documentation/pull/1519

was replaced with this one:

https://github.com/temporalio/documentation/pull/1520

due to Netlify being part of CI and requiring the PR creator to have a Netlify seat on Temporal's team.

Solution

Ideally in an OSS project, we're able to merge community contributions. A couple ideas:

  • Change Netlify setup so that it doesn't require PR creator to be a user (I haven't looked into this—maybe a Netlify setting, or using a GH Action yaml file instead of Netlify's Action?)
  • Make Netlify not a required CI step.

lorensr avatar Aug 30 '22 23:08 lorensr