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

Reorganized Nimble source structure. Fixes #267 #274

Merged
merged 1 commit into from
Apr 1, 2016

Conversation

jeffh
Copy link
Member

@jeffh jeffh commented Mar 27, 2016

Bug: Cocoapods podspec wrongly included a Linux-only swift file. Added
exclude_files to cocoapod podspec.

This commit also reorganizes the structure of Nimble to avoid future
errors and simplify the groups a bit:

  • Objective-C specific code is now in Sources/Nimble/Adapters/ObjectiveC
  • Non-Objective-C specific code is now in Sources/Nimble/Adapters/NonObjectiveC
    • This is code that Xcode should ignore
  • Rename ObjC prefixed files to NMB
  • Move files in Wrapper to Matchers

Bug: Cocoapods podspec wrongly included a Linux-only swift file. Added
exclude_files to cocoapod podspec.

This commit also reorganizes the structure of Nimble to avoid future
errors and simplify the groups a bit:

 - Objective-C specific code is now in Sources/Nimble/Adapters/ObjectiveC
 - Non-Objective-C specific code is now in Sources/Nimble/Adapters/NonObjectiveC
   - This is code that Xcode should **ignore**
 - Rename `ObjC` prefixed files to `NMB`
 - Move files in `Wrapper` to `Matchers`
@briancroom
Copy link
Member

👍 thanks for fixing this up @jeffh!

@briancroom briancroom merged commit d4cbf8d into Quick:master Apr 1, 2016
Megal pushed a commit to Megal/Nimble that referenced this pull request Jul 31, 2019
Reorganized Nimble source structure. Fixes Quick#267
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