Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DOCS] Skip docs deploy on forks #1351

Merged
merged 1 commit into from
Apr 22, 2024

Conversation

jbampton
Copy link
Member

https://docs.github.com/en/actions/using-jobs/using-conditions-to-control-job-execution#example-only-run-job-for-specific-repository

Did you read the Contributor Guide?

Is this PR related to a JIRA ticket?

  • No, this is a documentation update. The PR name follows the format [DOCS] my subject.

What changes were proposed in this PR?

When a contributor syncs their fork from the upstream and pushes back to origin master on GitHub the docs build currently fails as seen in the next two links:

https://github.com/jbampton/sedona/actions/runs/8780798846/job/24091461356

https://github.com/furqaankhan/sedona/actions/runs/8725430958/job/23938384642

How was this patch tested?

Here is an image of the failure:

Screenshot from 2024-04-22 19-02-05

Probably the worst part about this bug is that you get sent out an email from GitHub about the failing Docs build.

Just extra email spam like this:

Screenshot from 2024-04-22 18-38-30

Did this PR include necessary documentation updates?

  • No, this PR does not affect any public API so no need to change the documentation.

@jiayuasu jiayuasu merged commit 4cc528b into apache:master Apr 22, 2024
3 checks passed
@jbampton jbampton deleted the skip-docs-deploy-on-forks branch April 22, 2024 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants