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

Fix injecting Swift classes #120

Merged
merged 8 commits into from
Mar 27, 2019
Merged

Fix injecting Swift classes #120

merged 8 commits into from
Mar 27, 2019

Conversation

zenangst
Copy link
Collaborator

@zenangst zenangst commented Mar 27, 2019

  • Adds print statement for when code coverage needs to be disabled 4eec370
  • Refactors the regular expression when grep-ing for classes when injecting Swift classes dac61ec
  • Adds a new condition for checking if an injected class is a Swift class 2741fcd

zenangst and others added 8 commits March 27, 2019 14:15
This is done to match the new format when grep-ing classes.
Add new condition to match swift classes.

Co-Authored-By: John Holdsworth <johnno1962@users.noreply.github.com>
Co-Authored-By: John Holdsworth <johnno1962@users.noreply.github.com>
Co-Authored-By: John Holdsworth <johnno1962@users.noreply.github.com>
@zenangst zenangst changed the title Fix injecting swift classes Fix injecting Swift classes Mar 27, 2019
Copy link
Owner

@johnno1962 johnno1962 left a comment

Choose a reason for hiding this comment

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

You even fixed the README 🙂

@zenangst zenangst merged commit f05e96c into master Mar 27, 2019
@zenangst zenangst deleted the fix/injecting-swift-classes branch March 27, 2019 21:15
@su350380433 su350380433 mentioned this pull request Apr 19, 2019
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