Closed praneelc closed 4 years ago
@trang1618 if contributor make a PR from a forked repo, the 1st CI check should failed (like this) maybe we should remove any git push
or deploy step in CI for PR.
if contributor make a PR from a forked repo, the 1st CI check should failed (like this) maybe we should remove any git push or deploy step in CI for PR.
There should be a way around this. The reason I wanted to generate these files in PR is to avoid adding anything straight to master without reviewing. Let's explore our options a bit...
Another option is to trigger the build on push events of the forked repo.
I believe this has now been fixed in #130.
@praneelc Could I ask you to re-submit this PR one more time, please? Thank you for being patient with us as we figure out the workflow.
resubmitting as PR to master branch