mapbox / mapbox/mapbox-plugins-android

Fix CI for external contributed PRs

Open
#995 2 comments 1 reaction 0 assignees View on GitHub
bug build
Dominant language
Java
Stars
242
Forks
116
PR merge metrics
No merged PRs in 30d

Description

Our CI workflow doesn't expose env. variables for security reasons when running PRs coming from forked repositories. We need to optimise our workflow to not execute steps that require them. Example of failed build [here](https://circleci.com/gh/mapbox/mapbox-plugins-android/3324?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link).

cc @mapbox/maps-android

Contributor guide

Open the contributing guide

Research direction

Start with the CI workflow and the failed CircleCI build linked in the issue. Identify steps that require environment variables unavailable to forked PRs; done means external contributed PRs skip those steps and the workflow completes successfully.

Written by the indexing model from the issue text.

Assessment

Domain
ci-cd
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.