Skip to content

[AMSDK-10442] Cleanup project structure part2#67

Merged
emdobrin merged 4 commits intoadobe:devfrom
kevinlind:amsdk-10442-part2
Sep 28, 2020
Merged

[AMSDK-10442] Cleanup project structure part2#67
emdobrin merged 4 commits intoadobe:devfrom
kevinlind:amsdk-10442-part2

Conversation

@kevinlind
Copy link
Copy Markdown
Contributor

Description

Move/Rename source and test code folders to fall closer in line with Swift project structures. Update Xcode projects, scripts, and Podfiles with new folder names.

  • Rename folder 'demo' to 'SampleApps'
  • Rename 'code/src' to 'Sources'
  • Rename 'code/unitTests' to 'Tests/UnitTests'
  • Rename 'code/functionalTests' to 'Tests/FunctionalTests'
  • Rename 'code/testUtilShared' to 'Tests/TestUtils'

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Rename 'code/unitTests' to 'Tests/UnitTests', 'code/functionalTests' to 'Tests/FunctionalTests', and 'code/testUtilsShared' to 'Tests/TestUtils'. Rename Xcode test targets and schemes to UnitTests and FunctionalTests. Update scripts and Podfile with the renames.
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 28, 2020

Codecov Report

Merging #67 into dev will decrease coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev      #67      +/-   ##
==========================================
- Coverage   88.11%   87.96%   -0.15%     
==========================================
  Files          17       17              
  Lines         555      573      +18     
==========================================
+ Hits          489      504      +15     
- Misses         66       69       +3     

- SampleApps/*/Pods
- SampleApps/*/xdmlib
- SampleApps/*/libs
- Pods
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@emdobrin emdobrin merged commit 5f79557 into adobe:dev Sep 28, 2020
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