Skip to content

Conversation

@Datron
Copy link
Collaborator

@Datron Datron commented Dec 8, 2025

Problem

Missing provider docs

Solution

Added docs for setup and usage of provider

@Datron Datron requested a review from a team as a code owner December 8, 2025 10:56
@semanticdiff-com
Copy link

semanticdiff-com bot commented Dec 8, 2025

Review changes with  SemanticDiff

@Datron Datron force-pushed the haskell-of-example branch 2 times, most recently from b5b4efb to cfa7a73 Compare December 9, 2025 13:15
@knutties knutties changed the title chore(docs): add documentation for setup and usage of haskell provider docs(haskell): add documentation for setup and usage of haskell provider Dec 11, 2025
inputs.superposition.haskellFlakeProjectModules.output
];

basePackages = pkgs.haskell.packages.ghc96;
Copy link
Collaborator

Choose a reason for hiding this comment

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

The development shell uses GHC 9.4 ( at line number 67) while the project is configured for GHC 9.6. This can cause confusing build failures. Can we check ?

Add the following dependencies to your .cabal file:

```cabal
monad-logger-aeson == 0.4.1.3,
Copy link
Collaborator

Choose a reason for hiding this comment

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

It is too strict , Can we change this to : monad-logger-aeson ^>= 0.4.1

threadDelay 4000000
OpenFeature.setDefaultProvider provider
client <- OpenFeature.createClient
v <- expectRight <$> OpenFeature.getBoolValue client (pack "k1") mempty
Copy link
Collaborator

Choose a reason for hiding this comment

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

The example uses "k1" but doesn't explain what flag keys should exist

```bash
cd /tmp

curl -L "https://github.com/juspay/superposition/releases/download/{version-you-want-to-use}/superposition_core-${1}.zip" \
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we add the v prefix here as well ?

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