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.

Sign up for GitHub

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

Jump to bottom

Tidying #431

Merged
merged 6 commits into from
Jun 12, 2023
Merged

Tidying #431

merged 6 commits into from
Jun 12, 2023

Conversation

ZevEisenberg
Copy link
Collaborator

@ZevEisenberg ZevEisenberg commented Jun 12, 2023 •
edited
Loading

Update gems and clean up some less-than-ideal asserts.

eliotw1
eliotw1 approved these changes Jun 12, 2023
View reviewed changes
Copy link
Contributor

@eliotw1 eliotw1 left a comment

Choose a reason for hiding this comment

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

👍

@raizlabs-oss-bot
Copy link
Collaborator

raizlabs-oss-bot commented Jun 12, 2023 •
edited
Loading

2 Messages
📖 Test Results
📖 Code Coverage

Current coverage for BonMot.framework is 77.50%

Files changed - -
StringStyle+Part.swift 99.23% ✅

Powered by xcov

Generated by 🚫 Danger

@ZevEisenberg ZevEisenberg merged commit 001139a into master Jun 12, 2023
@ZevEisenberg ZevEisenberg deleted the tidying branch June 12, 2023 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eliotw1 eliotw1 eliotw1 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ZevEisenberg @raizlabs-oss-bot @eliotw1