[WIP] [Testing Snapshot] [DO NOT MERGE] GHC 9.2.7 upgrade testing-snapshot diff #47
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Since We updated Euler-hs versions, some of the functions it used to export are not there anymore. * Updates the code to reflect the new state of Euler-hs exports.
* ManagerSelector type in an older version of euler used to be a type synonym for "Text". * This is now a newtype with a newtype constructor "ManagerSelector" from Euler.Types modules * Wraps old useages with newtype constructor for correct type.
* The types changed in Aeson2 and beckn-shared-kernel * Adds unordered-containers package in app/gateway's package.yaml to get the "HashMap" container. * Changes types to "HashMap Text" where needed from "Map String"
…nges * Removes now redundant imports after making the Aeson2 and beckn-shared-kernel changes in the previous commit
* makes "ambiguous-fields" a warning instead of an error.
* Adds the updated .cabal files for beckn-gateway and mock-registry. * Updates the cabal files with the changes done in package.yaml files in the previous commits * The cabal files were recently commited in the repository and are auto-generated by hpack tool, so we need to update them as we go.
* Updates flake.nix file with the following changes - update common repo to one that provides GHC 9.2.7 package set - updates shared-kernel to one that builds with GHC 9.2.7 * Updates the corresponding flake.lock file with the above changes.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Type of Change
Description
This is a dummy PR for GHC 9.2.7 upgrade changes from the testing-snapshot branch. This is not the final version and is intended as a preview before the final PRs are up. These changes enable the project to be built with GHC 9.2.7 and the upgraded libraries & dependencies.
Most commits should have detailed contextual point-based descriptions of what changed and why with that commit for much more convenient understanding of the changes made with that commit.
Additional Changes
Motivation and Context
This was done as part of upgrading the entire mobility set of projects to GHC 9.2.7
How did you test it?
This repo does not have any tests to run. The project compiling and building successfully was thus taken as a proxy for now.
Full ride flow onceover testing and basic sanity testing was done on the nammayatri repo, of which, this is a dependency.