Skip to content

Conversation

arriolac
Copy link
Contributor

@arriolac arriolac commented Jun 3, 2022

Adding secrets-gradle-plugin to handle reading key/values from local.properties.

@arriolac arriolac requested a review from a team as a code owner June 3, 2022 20:41
@arriolac arriolac requested review from simona-anomis and riggaroo and removed request for simona-anomis June 3, 2022 20:41
# isn't picking up the key when creating a local.properties file here
sed -i -e "s,\${googleMapsKey},$GOOGLE_MAPS_KEY,g" ${{ env.SAMPLE_PATH }}/app/src/main/AndroidManifest.xml
env:
GOOGLE_MAPS_KEY: ${{ secrets.GOOGLE_MAPS_KEY }}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great - Would it also fix #746 ? CityMapViewTests

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This uses the same API key so if the issue is due to key restrictions this won't fix it. I can take a look at #746 though to see what's going on

@arriolac arriolac merged commit 33a3861 into main Jun 9, 2022
@arriolac arriolac deleted the chris/crane/secrets-gradle branch June 9, 2022 16:07
@riggaroo riggaroo mentioned this pull request Jun 15, 2022
4 tasks
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.

2 participants