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

[Chore] Update some dependencies #145

Merged
merged 8 commits into from
Jan 22, 2025
Merged

Conversation

gromakovsky
Copy link
Member

@gromakovsky gromakovsky commented Jan 20, 2025

Description

This PR bumps upper bounds on some deps and updates tooling versions in CI config.

Related issue(s)

None

✅ Checklist for your Pull Request

Ideally a PR has all of the checkmarks set.

If something in this list is irrelevant to your PR, you should still set this
checkmark indicating that you are sure it is dealt with (be that by irrelevance).

Related changes (conditional)

  • Tests

    • If I added new functionality, I added tests covering it.
    • If I fixed a bug, I added a regression test to prevent the bug from
      silently reappearing again.
  • Documentation

    • I checked whether I should update the docs and did so if necessary:
  • Public contracts

    • Any modifications of public contracts comply with the Evolution
      of Public Contracts
      policy.
    • I added an entry to the changelog if my changes are visible to the users
      and
    • provided a migration guide for breaking changes if possible

Stylistic guide (mandatory)

✓ Release Checklist

  • I updated the version number in o-clock.cabal.
  • I updated the changelog and moved everything
    under the "Unreleased" section to a new section for this release version.
  • If any definitions (functions, type classes, instances, etc) were added,
    I added @since haddock annotations.
  • (After merging) I created a new entry in the releases page,
    with a summary of all user-facing changes.
    • I made sure a tag was created using the format vX.Y.Z
  • (After merging) I uploaded the package to hackage.

The following command passes:
> cabal test --constraint 'hedgehog==1.5'  --constraint 'doctest==0.23.0'
haskell/actions/setup was deprecated and suggests using haskell-actions/setup.
@gromakovsky gromakovsky force-pushed the gromak/maintenance-jan25 branch from a010c72 to 1c100bb Compare January 22, 2025 15:50
@gromakovsky gromakovsky requested a review from int-index January 22, 2025 15:50
Copy link
Member

@int-index int-index left a comment

Choose a reason for hiding this comment

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

LGTM except for the two minor suggestions

o-clock.cabal Outdated Show resolved Hide resolved
@@ -72,7 +72,9 @@ import Control.Monad.IO.Class (MonadIO, liftIO)
import Data.Char (isDigit, isLetter)
import Data.Coerce (coerce)
import Data.Data (Data)
#if ( __GLASGOW_HASKELL__ < 910 )
Copy link
Member

Choose a reason for hiding this comment

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

This conditional is related to library changes, not language changes, so it would be more appropriate to check it with #if !(MIN_VERSION_base(4,20,0))

Problem: foldl' is exported from Prelude starting from
ghc-9.10 (base 4.20), so explicitly importing it causes a warning.

Solution: use CPP to import it only for base older than 4.20.
Problem: with recent ghc readme-test complains about orphan instances
in README.md.

Solution: these instances are intentionally defined in README, so we
supress the warning.
There are some minor changes in the package, so we're bumping
the patch version.
@gromakovsky gromakovsky force-pushed the gromak/maintenance-jan25 branch from 9d44b65 to 52385db Compare January 22, 2025 17:57
@gromakovsky gromakovsky merged commit 7261738 into master Jan 22, 2025
12 checks passed
@gromakovsky gromakovsky deleted the gromak/maintenance-jan25 branch January 22, 2025 18:10
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