Skip to content

chore: remove unused google-services-file from POS workflows#360

Merged
ignaciosantise merged 1 commit intomainfrom
feat/remove-google-services-pos-actions
Jan 26, 2026
Merged

chore: remove unused google-services-file from POS workflows#360
ignaciosantise merged 1 commit intomainfrom
feat/remove-google-services-pos-actions

Conversation

@ignaciosantise
Copy link
Copy Markdown
Collaborator

Summary

Remove unused google-services-file secrets from the iOS release workflows for both pos-app and poc-pos-app. These secrets were being passed to the base iOS workflow but were never declared or used there, making them dead code that could be safely removed.

Changes

  • Removed POS_IOS_GOOGLE_SERVICES_FILE_BASE64 from .github/workflows/release-pos.yaml
  • Removed POC_POS_IOS_GOOGLE_SERVICES_FILE_BASE64 from .github/workflows/release-pos-poc.yaml

Test Plan

  • Verify that the next POS release workflows complete successfully without these unused secrets

The google-services-file secret was being passed to release-ios-base.yaml but that workflow never declared or used this secret, making it dead code. Remove from both pos-app and poc-pos-app iOS release configurations.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings January 26, 2026 17:26
@vercel
Copy link
Copy Markdown

vercel bot commented Jan 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
react-native-examples Ready Ready Preview, Comment Jan 26, 2026 5:27pm

Request Review

@claude
Copy link
Copy Markdown

claude bot commented Jan 26, 2026

Claude finished @ignaciosantise's task in 38s —— View job


Review Complete

  • Read modified workflow files
  • Verify secrets are unused
  • Check for other references
  • Provide review feedback

✅ No issues found

Verified:

  • Base iOS workflow (.github/workflows/release-ios-base.yaml:61-88) declares no google-services-file secret parameter
  • Removed secrets were passed to base workflow but never consumed
  • No other references to these secrets in codebase
  • Safe cleanup of dead code

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes unused Google services-related secrets from the POS iOS release workflows now that the base iOS workflow does not declare or consume them.

Changes:

  • Removed the POS_IOS_GOOGLE_SERVICES_FILE_BASE64 secret forwarding from .github/workflows/release-pos.yaml.
  • Removed the POC_POS_IOS_GOOGLE_SERVICES_FILE_BASE64 secret forwarding from .github/workflows/release-pos-poc.yaml.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/release-pos.yaml Stops passing an undeclared/unused google-services-file secret into the base iOS workflow for the POS app.
.github/workflows/release-pos-poc.yaml Stops passing an undeclared/unused google-services-file secret into the base iOS workflow for the POC POS app.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ignaciosantise ignaciosantise merged commit e2bdab9 into main Jan 26, 2026
17 checks passed
@ignaciosantise ignaciosantise deleted the feat/remove-google-services-pos-actions branch January 26, 2026 17:57
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