Skip to content

Migrate to shared 1Password setup action - #2413

Merged
roryabraham merged 1 commit into
mainfrom
update-1password-cli-action
Dec 16, 2025
Merged

Migrate to shared 1Password setup action#2413
roryabraham merged 1 commit into
mainfrom
update-1password-cli-action

Conversation

@roryabraham

Copy link
Copy Markdown
Contributor

Explanation of Change

This PR removes the duplicate local setup-certificate-1p composite action and migrates all usages to the new shared action in GitHub-Actions. The shared action uses an updated commit hash (9a0c9dd934086b7ab1d90115d455bda1c53c2bdb) that fixes an intermittent reliability issue in 1password/install-cli-action.

Hold for Expensify/GitHub-Actions#48

Fixed Issues

$ 1Password/install-cli-action#13

Auth / Web / Mobile Tests

  • [PR Author] I made sure this PR correctly handles default values

Automated tests

  • Ran actionlint, validateImmutableActionRefs, and validateWorkflowSchemas - all pass

Manual tests

  • No functional changes - this is a refactor to use a shared action with the same behavior
  • Verified workflow syntax is valid and action references use immutable commit hashes

- Use commit 9a0c9dd934086b7ab1d90115d455bda1c53c2bdb which fixes intermittent reliability issue
- Remove duplicate local action and use shared version from GitHub-Actions
@roryabraham roryabraham self-assigned this Dec 16, 2025
@roryabraham roryabraham changed the title [HOLD] Migrate to shared 1Password setup action Migrate to shared 1Password setup action Dec 16, 2025
@roryabraham
roryabraham merged commit f5f8ba1 into main Dec 16, 2025
12 of 13 checks passed
@roryabraham
roryabraham deleted the update-1password-cli-action branch December 16, 2025 23:15
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