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

[ADP-3378] Bump node to 8.12 #4657

Merged
merged 4 commits into from
Jul 10, 2024
Merged

[ADP-3378] Bump node to 8.12 #4657

merged 4 commits into from
Jul 10, 2024

Conversation

Anviking
Copy link
Member

@Anviking Anviking commented Jun 28, 2024

  • Bump runtime- and compile-time dependencies to 8.12

Comments

Issue Number

ADP-3381

@Anviking Anviking self-assigned this Jun 28, 2024
@Anviking Anviking changed the title anviking/bump node to 8.12 Bump node to 8.12 Jun 28, 2024
@Anviking Anviking force-pushed the anviking/bump-node-to-8.12 branch 2 times, most recently from 87997b6 to 8a7d8d4 Compare June 29, 2024 08:05
@Anviking Anviking force-pushed the anviking/bump-node-to-8.12 branch from 2535586 to f071494 Compare July 9, 2024 10:24
Comment on lines -81 to -88
TranslationLogicMissingInput inp ->
apiError err400 UnresolvedInputs
$ T.unwords
[ "The transaction I was given contains inputs I don't know"
, "about. Please ensure all foreign inputs are specified as "
, "part of the API request. The unknown input is:\n\n"
, T.pack $ show inp
]
Copy link
Member Author

Choose a reason for hiding this comment

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

I think this is basically unreachable anyway.

Comment on lines -108 to -114
apiError err400 TranslationByronTxOutInContext
$ T.unwords
[ "The transaction I was given contains a Byron-style TxOut"
, "which is not supported when executing Plutus scripts. "
, "The offending TxOut is:\n\n"
, T.pack $ show txOut
]
Copy link
Member Author

Choose a reason for hiding this comment

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

This is not unreachable, but i'd question the worth of maintaining this error mapping now. I don't think anyone uses it.

@Anviking Anviking force-pushed the anviking/bump-node-to-8.12 branch from f071494 to affb8f5 Compare July 9, 2024 10:39
@Anviking Anviking marked this pull request as ready for review July 9, 2024 10:50
@Anviking Anviking requested a review from paolino July 9, 2024 10:50
@@ -24,6 +24,7 @@
{- HLINT ignore "Use camelCase" -}

{-# OPTIONS_GHC -fno-warn-orphans #-}
{-# OPTIONS_GHC -fno-warn-deprecations #-}
Copy link
Member Author

Choose a reason for hiding this comment

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

Can be addressed in later PR IMO.

cabal.project Outdated Show resolved Hide resolved
@Anviking Anviking force-pushed the anviking/bump-node-to-8.12 branch 2 times, most recently from 1f29ad3 to 96a4f97 Compare July 9, 2024 12:06
@@ -147,6 +147,7 @@ constructAddressFromIx role pTemplate dTemplate ix =
Nothing ->
createEnterpriseAddress pScript

-- | NOTE: The roles 'DRep', 'CCCold', 'CCHot' are not supported.
Copy link
Member Author

Choose a reason for hiding this comment

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

Although, I think restricting the domain with a new type containing a subset of CA.Role should be fairly painless. May try...

@paolino paolino changed the title Bump node to 8.12 [ADP-3378] Bump node to 8.12 Jul 9, 2024
Copy link
Collaborator

@paolino paolino left a comment

Choose a reason for hiding this comment

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

LGTM

@Anviking Anviking force-pushed the anviking/bump-node-to-8.12 branch from 96a4f97 to 1be0fb9 Compare July 9, 2024 15:38
@Anviking Anviking enabled auto-merge July 9, 2024 15:51
@Anviking Anviking added this pull request to the merge queue Jul 9, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 9, 2024
@Anviking Anviking force-pushed the anviking/bump-node-to-8.12 branch 3 times, most recently from ad8cc9f to c884616 Compare July 9, 2024 20:49
Anviking added 3 commits July 10, 2024 11:42
- Adjust versions of tree-diff, ansi-wl-pprint, bech32
- wip: fix compilation errors
- Use cardano-addresses with optparse-applicative 0.18.1
- Compilation fixes in BalanceSepec
- Fixes in `Cardano.Api.Gen`
- fixes for balance-tx tests
- Fix and simplify `ErrAssignReedemers`
- Regenerate Wallet CLI goldens
- fixup replaceCosignersWithVerKeys
…ers`

Proper fix can go in another PR and task.
@Anviking Anviking force-pushed the anviking/bump-node-to-8.12 branch from c884616 to 6758721 Compare July 10, 2024 09:42
@Anviking Anviking enabled auto-merge July 10, 2024 10:02
@Anviking Anviking added this pull request to the merge queue Jul 10, 2024
Merged via the queue into master with commit 1b8c38a Jul 10, 2024
3 checks passed
@Anviking Anviking deleted the anviking/bump-node-to-8.12 branch July 10, 2024 10:58
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