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

GitHub Actions Workflow doesn't work correctly for PR from forks #109

Closed
CAMOBAP opened this issue Apr 18, 2023 · 1 comment
Closed

GitHub Actions Workflow doesn't work correctly for PR from forks #109

CAMOBAP opened this issue Apr 18, 2023 · 1 comment
Assignees

Comments

@CAMOBAP
Copy link
Collaborator

CAMOBAP commented Apr 18, 2023

Problem

https://github.com/hCaptcha/hcaptcha-android-sdk/actions/runs/4703861808/jobs/8394211306

GitHub Actions Workflow doesn't work correctly for PR from forks, it fails on:

Sonar Job:

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':sdk:sonarqube'.
> Project not found. Please check the 'sonar.projectKey' and 'sonar.organization' properties, the 'SONAR_TOKEN' environment variable, or contact the project administrator

Diffuse Job:

Error: Resource not accessible by integration
Error: See this action's readme for details about this error

Proposed solution

Check those links to find/implement a solution:

@CAMOBAP CAMOBAP self-assigned this Apr 18, 2023
CAMOBAP added a commit that referenced this issue Jul 16, 2024
* ci: skip sonarqube step for PR from form #109

* ci: use fresh AVD from benchmark tests (#157)

* refactor: build gradle (#161)

* chore: split big sdk/build.gradle to several applied gradle sripts

* chore: upload all outputs and reports on benchmark failure

* ci: move timeout restoriction to step level and reduce it to 20

* debug: enable artifact upload for success bench

* fix: ignore HCaptchaWebViewHelperTest.benchmarkWebViewLoad benchmark

* ci: skip sonarqube step for PR from form #109

* ci: migrate to own check-user-permission action

* fix: bad prior merge

* fix: add value to outputs of check-user-permission action
@CAMOBAP
Copy link
Collaborator Author

CAMOBAP commented Jul 16, 2024

A new approach is implemented. Will be closed after test on some fork-based PR

@CAMOBAP CAMOBAP closed this as completed Oct 30, 2024
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

No branches or pull requests

1 participant