Skip to content

Conversation

@heckj
Copy link
Contributor

@heckj heckj commented Mar 1, 2023

I have a dead-simple linear bucket histogram I'm using in an existing app, and I'd very much like to replace it with this HdrHistogram implementation - but I still need to back-support to Catalina for that app. I checked this package locally, and lowering the versions doesn't impact anything currently - but I realize there's more development that you have in mind here. So... I'm opening this PR to downgrade the required versions of Swift, and related OS pieces, for your consideration.

Description

  • reduces swift required to build this package to Swift 5.6
  • reduces macOS support back to allow/support Catalina (10.15)
  • adds explicit iOS and watchOS support

How Has This Been Tested?

swift build && swift test

Minimal checklist:

  • I have performed a self-review of my own code
  • I have added DocC code-level documentation for any public interfaces exported by the package
  • I have added unit and/or integration tests that prove my fix is effective or that my feature works

- reduces swift required to build this package to Swift 5.6
- reduces macOS support back to allow/support Catalina (10.15)
- adds explicit iOS and watchOS support
@codecov
Copy link

codecov bot commented Mar 2, 2023

Codecov Report

Merging #8 (63caea1) into main (604410b) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main       #8   +/-   ##
=======================================
  Coverage   98.10%   98.10%           
=======================================
  Files           4        4           
  Lines        1631     1631           
=======================================
  Hits         1600     1600           
  Misses         31       31           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 604410b...63caea1. Read the comment docs.

@hassila hassila requested a review from dimlio March 2, 2023 09:30
Copy link
Collaborator

@dimlio dimlio left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@dimlio dimlio merged commit e9214c5 into HdrHistogram:main Mar 7, 2023
@heckj heckj deleted the lowering_req_swift branch March 7, 2023 14:55
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