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 Tests directory structure #135

Merged
merged 1 commit into from
Apr 5, 2021
Merged

Update Tests directory structure #135

merged 1 commit into from
Apr 5, 2021

Conversation

mac-gallagher
Copy link
Owner

@mac-gallagher mac-gallagher commented Apr 5, 2021

  • Update Tests/ directory structure
  • Rename test files from *Spec to *Test
  • Add back iPad deployments for ShuffleExample (removed accidentally)
  • Deleted duplicate CONTRIBUTING.md file under Documentation/
  • Prefer UIKit imports over Foundation and CoreGraphics to avoid Swift PM build issues (from similar past experiences)
  • Fix some indentation issues

@codecov-io
Copy link

Codecov Report

Merging #135 (da2a5b3) into master (90adc0e) will not change coverage.
The diff coverage is n/a.

❗ Current head da2a5b3 differs from pull request most recent head 298e096. Consider uploading reports for the commit 298e096 to get more accurate results

@@          Coverage Diff           @@
##           master    #135   +/-   ##
======================================
  Coverage    84.8%   84.8%           
======================================
  Files          13      13           
  Lines         759     759           
======================================
  Hits          644     644           
  Misses        115     115           
Impacted Files Coverage Δ
...ses/SwipeCardStack/CardStackAnimationOptions.swift 100.0% <ø> (ø)
...fle/Classes/SwipeCardStack/CardStackAnimator.swift 0.0% <ø> (ø)
...Classes/SwipeCardStack/CardStackStateManager.swift 100.0% <ø> (ø)

@mac-gallagher mac-gallagher merged commit 2d7e372 into master Apr 5, 2021
@mac-gallagher mac-gallagher deleted the tests branch April 5, 2021 05: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.

2 participants