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

Add check to publish only when repository is "Qbeast-io/qbeast-spark" #60

Merged
merged 1 commit into from
Dec 22, 2021

Conversation

eavilaes
Copy link
Contributor

Adding a simple check for the repository name to avoid running the sbt publish on the repository forks' CI. The syntax used is the one from GitHub Actions Quickstart.

Fixes #52

@eavilaes eavilaes self-assigned this Dec 22, 2021
@eavilaes eavilaes requested review from cugni and osopardo1 December 22, 2021 13:18
@osopardo1
Copy link
Member

Seems good to me!

@eavilaes eavilaes merged commit 35c9f56 into Qbeast-io:main Dec 22, 2021
@eavilaes
Copy link
Contributor Author

I can confirm it has been fixed. Publish step is not being executed on forks:
https://github.com/eavilaes/qbeast-spark/runs/4607461599?check_suite_focus=true

@eavilaes eavilaes deleted the 52-fix-ci branch December 22, 2021 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI failing on repository Forks, due to GitHub Package Registry key missing on them
2 participants