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

refactor: moved public pagination structures to directory API #378

Merged
merged 2 commits into from
May 12, 2024

Conversation

philprime
Copy link
Member

No description provided.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@philprime philprime self-assigned this May 12, 2024
@techprimate-bot
Copy link

Fails
🚫 Added or removed library files require the Carthage Xcode project to be updated.
Warnings
⚠️ Any changes to library code should be reflected in the Changelog. Please consider adding a note there.
⚠️ The library files were changed, but the tests remained unmodified. Consider updating or adding to the tests to match the library changes.

SwiftLint found issues

Severity File Reason
Warning PDFPaginationStyle.swift:13 Line should be 150 characters or less: currently 151 characters (line_length)

Generated by 🚫 Danger Swift against 0fc45fd

WIP

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@techprimate-bot
Copy link

Warnings
⚠️ Any changes to library code should be reflected in the Changelog. Please consider adding a note there.
⚠️ The library files were changed, but the tests remained unmodified. Consider updating or adding to the tests to match the library changes.

SwiftLint found issues

Severity File Reason
Warning PDFPaginationStyle.swift:13 Line should be 150 characters or less: currently 151 characters (line_length)

Generated by 🚫 Danger Swift against 79a3a2b

@kodiakhq kodiakhq bot merged commit 0230f73 into main May 12, 2024
7 checks passed
@kodiakhq kodiakhq bot deleted the docs/updated-pagination-docs branch May 12, 2024 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants