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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Foundation API for Base64 Encoding #403

Merged

Conversation

Mazyod
Copy link
Contributor

@Mazyod Mazyod commented May 1, 2019

Note, I didn't see the need to update the changelog, as this is extremely minor and should be transparent to the users. If you'd like an entry anyway, I can add that as well.

Fixes #394.

# Conflicts:
#	Sources/HttpParser.swift
#	XCode/Sources/String+BASE64.swift
@swifter-bot
Copy link

swifter-bot commented May 1, 2019

2 Warnings
⚠️ It seems like you’ve added new tests to the library. If that’s the case, please update the XCTestManifests.swift file running in your terminal the command swift test --generate-linuxmain.
⚠️ If you ran the command swift test --generate-linuxmain in your terminal, please remove the line testCase(IOSafetyTests.__allTests__IOSafetyTests), from public func __allTests() -> [XCTestCaseEntry] in the bottom of the file. For more reference see #366.
1 Message
📖 Hey, @Mazyod 👋.

Generated by 🚫 Danger

@Vkt0r
Copy link
Member

Vkt0r commented May 1, 2019

@Mazyod Danger would require it 😅

@Vkt0r
Copy link
Member

Vkt0r commented May 1, 2019

This is weird I have to rebuild the Linux job the PR three times in CircleCI regarding issues with the Docker image and your PR was working from me in Linux with the same Docker image locally for me since the beginning 🤔. Anyways I would file a report with them for investigation.

@Vkt0r Vkt0r self-requested a review May 1, 2019 04:48
Copy link
Member

@Vkt0r Vkt0r left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thanks again for your contribution.

@Vkt0r Vkt0r merged commit d3c2d21 into httpswift:stable May 1, 2019
@Mazyod Mazyod deleted the enhancement/foundation-base64-encoding branch May 1, 2019 05:25
tomieq pushed a commit to tomieq/swifterfork that referenced this pull request Apr 1, 2021
…ase64-encoding

Use Foundation API for Base64 Encoding
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.

Use Foundation Base64Encoding
3 participants