Skip to content

OnePassword adapter refactor#1584

Merged
djmb merged 1 commit intomainfrom
all-fields-one-password-refactor
Jun 17, 2025
Merged

OnePassword adapter refactor#1584
djmb merged 1 commit intomainfrom
all-fields-one-password-refactor

Conversation

@djmb
Copy link
Collaborator

@djmb djmb commented Jun 17, 2025

  • fix rubocop offenses
  • extract fields_map
  • no early return
  • include fields in error message

- fix rubocop offenses
- extract fields_map
- no early return
- include fields in error message
@djmb djmb merged commit 9244247 into main Jun 17, 2025
10 checks passed
@djmb djmb deleted the all-fields-one-password-refactor branch June 17, 2025 10:52
airblade added a commit to airblade/kamal that referenced this pull request Jun 20, 2025
This postfix conditional is left over from the refactor in basecamp#1584 and is no longer needed.
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.

1 participant