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

Long-overdue package cleanup #183

Merged
merged 3 commits into from
Aug 19, 2023
Merged

Long-overdue package cleanup #183

merged 3 commits into from
Aug 19, 2023

Conversation

gwynne
Copy link
Member

@gwynne gwynne commented Aug 19, 2023

  • Update minimum Swift version to match Vapor (5.6)
  • Fix up CI
  • Clean up README
  • Fix platform conditions
  • Remove obsolete Swift version conditions
  • Remove obsolete Concurrency guards (back-deployment works)
  • Add API docs customization

Gets a semver-minor bump due to the new Swift version requirement.

…, fix platform and Swift version conditions, remove Concurrency guards
@gwynne gwynne added enhancement New feature or request semver-minor When merged, a new minor version release will be generated labels Aug 19, 2023
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (main@05ebd3a). Click here to learn what that means.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #183   +/-   ##
=======================================
  Coverage        ?   49.14%           
=======================================
  Files           ?       43           
  Lines           ?     1703           
  Branches        ?        0           
=======================================
  Hits            ?      837           
  Misses          ?      866           
  Partials        ?        0           

@gwynne gwynne enabled auto-merge (squash) August 19, 2023 03:48
Copy link

@dannflor dannflor left a comment

Choose a reason for hiding this comment

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

Nice to see all those conditionals go away

@gwynne gwynne merged commit 9a12000 into main Aug 19, 2023
13 checks passed
@gwynne gwynne deleted the general-cleanup branch August 19, 2023 03:48
@penny-for-vapor
Copy link

These changes are now available in 4.7.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request semver-minor When merged, a new minor version release will be generated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants