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

Add support for Swift 3 ACLs (open & fileprivate) in existing rules #905

Merged
merged 4 commits into from Nov 29, 2016
Merged

Add support for Swift 3 ACLs (open & fileprivate) in existing rules #905

merged 4 commits into from Nov 29, 2016

Conversation

marcelofabri
Copy link
Collaborator

Fixes #781 and #831

@codecov-io
Copy link

codecov-io commented Nov 27, 2016

Current coverage is 86.06% (diff: 79.16%)

Merging #905 into master will decrease coverage by 0.03%

@@             master       #905   diff @@
==========================================
  Files           114        114          
  Lines          5058       5066     +8   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           4355       4360     +5   
- Misses          703        706     +3   
  Partials          0          0          

Powered by Codecov. Last update e63c6ce...d9287ba

@dzenbot
Copy link

dzenbot commented Nov 27, 2016

This is basically the only Swift 3 issue I've been facing lately. Would be great to get this in!
Nice work @marcelofabri

Copy link
Collaborator

@jpsim jpsim left a comment

Choose a reason for hiding this comment

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

👍 Thanks for doing this. I hope you don't mind the few edits I made to your branch.

@jpsim jpsim merged commit 8383086 into realm:master Nov 29, 2016
@dzenbot
Copy link

dzenbot commented Nov 29, 2016

🎉🎉

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

4 participants