Skip to content

New Cardano.Api.Query.Expr module#25

Merged
newhoggy merged 1 commit intomainfrom
newhoggy/new-Expr-module
Jun 15, 2023
Merged

New Cardano.Api.Query.Expr module#25
newhoggy merged 1 commit intomainfrom
newhoggy/new-Expr-module

Conversation

@newhoggy
Copy link
Contributor

Description

Changelog

- description: |
    Use query functions from the new export `Cardano.Api.Query.Expr` module instead of using
    the query data types directly. 
  compatibility: no-api-changes
  type: maintenance

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • The change log section in the PR description has been filled in
  • New tests are added if needed and existing tests are updated. These may include:
    • golden tests
    • property tests
    • roundtrip tests
    • integration tests
      See Runnings tests for more details
  • Any changes are noted in the CHANGELOG.md for affected package
  • The version bounds in .cabal files are updated
  • CI passes. See note on CI. The following CI checks are required:
    • Code is linted with hlint. See .github/workflows/check-hlint.yml to get the hlint version
    • Code is formatted with stylish-haskell. See .github/workflows/stylish-haskell.yml to get the stylish-haskell version
    • Code builds on Linux, MacOS and Windows for ghc-8.10.7 and ghc-9.2.7
  • Self-reviewed the diff

Note on CI

If your PR is from a fork, the necessary CI jobs won't trigger automatically for security reasons.
You will need to get someone with write privileges. Please contact IOG node developers to do this
for you.

@newhoggy newhoggy force-pushed the newhoggy/new-Expr-module branch from 011a167 to d2311e0 Compare June 15, 2023 01:35
@newhoggy newhoggy force-pushed the newhoggy/new-Expr-module branch from d2311e0 to c799e9e Compare June 15, 2023 02:12
@newhoggy newhoggy added this pull request to the merge queue Jun 15, 2023
Merged via the queue into main with commit 5e42a37 Jun 15, 2023
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.

3 participants