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

update bluecryptor to 1.0.2. add swift 4.1 support #73

Merged
merged 8 commits into from
Apr 30, 2018

Conversation

quanvo87
Copy link
Collaborator

Description

Motivation and Context

How Has This Been Tested?

Checklist:

  • I have submitted a CLA form
  • If applicable, I have updated the documentation accordingly.
  • If applicable, I have added tests to cover my changes.

@CLAassistant
Copy link

CLAassistant commented Apr 30, 2018

CLA assistant check
All committers have signed the CLA.

Package.swift Outdated
@@ -16,7 +16,7 @@ let package = Package(
// .package(url: /* package url */, from: "1.0.0"),
.package(url: "https://github.com/IBM-Swift/BlueSocket.git", .upToNextMinor(from: "0.12.0")),
.package(url: "https://github.com/IBM-Swift/BlueSSLService.git", .upToNextMinor(from: "0.12.0")),
.package(url: "https://github.com/IBM-Swift/BlueCryptor.git", .upToNextMinor(from: "0.8.0")),
.package(url: "https://github.com/IBM-Swift/BlueCryptor.git", .upToNextMinor(from: "1.0.0")),
Copy link
Contributor

Choose a reason for hiding this comment

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

You might like to consider loosening this to just .from("1.0.0")

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

How about upToNextMajor?

Copy link
Contributor

Choose a reason for hiding this comment

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

Same thing, I think.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oh okay. I'll do that. from just sounds a bit scary in that it would pick up a new major as well.

@codecov-io
Copy link

codecov-io commented Apr 30, 2018

Codecov Report

Merging #73 into master will increase coverage by 0.21%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
+ Coverage   87.53%   87.75%   +0.21%     
==========================================
  Files          12       12              
  Lines         634      645      +11     
==========================================
+ Hits          555      566      +11     
  Misses         79       79
Flag Coverage Δ
#SwiftSMTP 87.75% <100%> (+0.21%) ⬆️
Impacted Files Coverage Δ
Sources/SwiftSMTP/SMTP.swift 96% <ø> (ø) ⬆️
Sources/SwiftSMTP/Attachment.swift 97.84% <ø> (ø) ⬆️
Sources/SwiftSMTP/SMTPError.swift 21.42% <ø> (ø) ⬆️
Sources/SwiftSMTP/MailSender.swift 96.92% <ø> (ø) ⬆️
Sources/SwiftSMTP/SMTPSocket.swift 76.52% <100%> (+2.48%) ⬆️
Sources/SwiftSMTP/Mail.swift 98.38% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3958929...86e2bbd. Read the comment docs.

@quanvo87 quanvo87 merged commit b6ca1ea into master Apr 30, 2018
@quanvo87 quanvo87 deleted the update-bluecryptor branch April 30, 2018 18:42
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

4 participants