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

build(ci): fix ci not running for user-submitted PR #497

Merged

Conversation

AnonymousWP
Copy link
Member

- resources/assets/uad_lists.json was missing here: https://github.com/Universal-Debloater-Alliance/universal-android-debloater-next-generation/blob/main/.github/workflows/ci.yml#L11. As you can see, the CI is not running for user-submitted PRs: #492. This should definitely be running, as users also tend to sometimes make a mistake, therefore messing up the JSON.

This PR fixes that issue, so now it's running for every PR, not matter if it comes from our organization or not.

@AnonymousWP AnonymousWP added the bug Something isn't working label Jun 3, 2024
@AnonymousWP AnonymousWP requested a review from a team June 3, 2024 08:23
Copy link
Contributor

deepsource-io bot commented Jun 3, 2024

Here's the code health analysis summary for commits 5e2aae9..eff8796. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Rust LogoRust✅ SuccessView Check ↗
DeepSource Test coverage LogoTest coverage⚠️ Artifact not reportedTimed out: Artifact was never reportedView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@adhirajsinghchauhan adhirajsinghchauhan merged commit c814057 into main Jun 3, 2024
2 of 3 checks passed
@adhirajsinghchauhan adhirajsinghchauhan deleted the ci/fix-workflow-not-running-in-certain-scenarios branch June 3, 2024 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants