Skip to content

Enhance EncryptionCipher with Identifiable and CaseIterable conformance, update version to 0.1.8#20

Merged
nedithgar merged 2 commits intomainfrom
feat/add-CaseIterable-and-Identifiable-to-EncryptionCipher
Oct 14, 2025
Merged

Enhance EncryptionCipher with Identifiable and CaseIterable conformance, update version to 0.1.8#20
nedithgar merged 2 commits intomainfrom
feat/add-CaseIterable-and-Identifiable-to-EncryptionCipher

Conversation

@nedithgar
Copy link
Copy Markdown
Owner

Add conformance to Identifiable and CaseIterable for the EncryptionCipher struct to improve usability in UI contexts. Update the version number to reflect the new changes.

Copilot AI review requested due to automatic review settings October 14, 2025 03:20
@nedithgar nedithgar merged commit 41ba541 into main Oct 14, 2025
4 checks passed
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

Add Identifiable and CaseIterable conformance to the EncryptionCipher struct to improve its usability in UI contexts, and update the CLI version to 0.1.8.

  • Added Identifiable conformance with id property returning the cipher's wire-format name
  • Added CaseIterable conformance with allCases property mirroring the existing known ciphers
  • Updated version number from 0.1.7 to 0.1.8

Reviewed Changes

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

File Description
Sources/SwiftKeyGenCLI/main.swift Version bump to 0.1.8
Sources/SwiftKeyGen/Formats/OpenSSH/OpenSSHPrivateKey.swift Enhanced EncryptionCipher with protocol conformances and documentation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@nedithgar nedithgar deleted the feat/add-CaseIterable-and-Identifiable-to-EncryptionCipher branch October 14, 2025 03:21
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