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

Support swift 4.2 #4

Merged
merged 5 commits into from Sep 24, 2018
Merged

Conversation

Vyazovoy
Copy link
Contributor

No description provided.

`WARN  | xcodebuild:  Quick/Sources/Quick/Behavior.swift:7:21: warning: static declarations are implicitly 'final'; use 'public' instead of 'open'`
@codecov-io
Copy link

Codecov Report

Merging #4 into master will increase coverage by 1.35%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   56.05%   57.41%   +1.36%     
==========================================
  Files          20       20              
  Lines         719      695      -24     
==========================================
- Hits          403      399       -4     
+ Misses        316      296      -20
Impacted Files Coverage Δ
Sources/UITableView/TableViewEditingDelegate.swift 0% <ø> (ø) ⬆️
Sources/UITableView/TableViewFactory.swift 0% <ø> (ø) ⬆️
Sources/UITableView/UITableView+Updates.swift 0% <0%> (ø) ⬆️
Sources/UITableView/TableViewDataSource.swift 0% <0%> (ø) ⬆️
Sources/DataSource.swift 90.9% <0%> (-1.4%) ⬇️
Sources/Utils/Diff.swift 10.71% <0%> (+2.81%) ⬆️

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 4fd0b61...13077b9. Read the comment docs.

- pod install --project-directory=Examples
- set -o pipefail && xcodebuild test -enableCodeCoverage YES -workspace Examples/SimpleSourceExample.xcworkspace -scheme SimpleSource-Unit-Tests -destination 'name=iPhone 8' ONLY_ACTIVE_ARCH=YES | xcpretty
- pod lib lint
- pod lib lint --allow-warnings
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Temporary change only to pass Quick related Xcode warning

@heiberg heiberg self-requested a review September 24, 2018 06:47
@heiberg heiberg self-assigned this Sep 24, 2018
@heiberg heiberg merged commit 4daf229 into Squarespace:master Sep 24, 2018
@heiberg
Copy link
Contributor

heiberg commented Sep 24, 2018

Splendid! Thanks a lot. 👍

@Vyazovoy Vyazovoy deleted the support-swift-4.2 branch September 24, 2018 23:01
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

3 participants