Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix typo in OneOfBuilder documentation #257

Merged
merged 1 commit into from Oct 26, 2022

Conversation

elfenlaid
Copy link
Contributor

Hello 馃憢 Hope all is well

The changes fix a small type for OneOfBuilder.swift documentation

I've took the example from its tests

final class OneOfBuilderTests: XCTestCase {
  func testBuildArray() {
    enum Role: String, CaseIterable, Equatable {
      case admin
      case guest
      case member
    }

    let parser = OneOf {
      for role in Role.allCases {
        role.rawValue.map { role }
      }
    }
    //...

Copy link
Member

@mbrandonw mbrandonw left a comment

Choose a reason for hiding this comment

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

Good catch, thanks!

@mbrandonw mbrandonw merged commit fb9949a into pointfreeco:main Oct 26, 2022
elfenlaid added a commit to elfenlaid/swift-parsing that referenced this pull request Oct 26, 2022
Hello 馃憢 

The changes fix a small type in `OneOfMany.swift` documentation

Sorry for missing it in [my previous PR](pointfreeco#257) 馃檱
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.

None yet

2 participants