Skip to content

Conversation

@OlaoluwaM
Copy link
Contributor

@OlaoluwaM OlaoluwaM commented May 12, 2025

Unofficial Haskell SDK for the cloudevents spec. Written with row types to allow for arbitrary extension attributes as per the spec requirements. This time without row-types.

This is a bare bones SDK that only allows for:

  • Creating a cloud event
  • Modifying a cloud event
  • Serializing a cloud event
  • Producing a Kafka message from a cloud event

Currently only missing field deduplication

Remove row-types

refactor: Slightly update some imports

build: Update extra-deps in stack lts yamls

Remove unsupported lts yamls

style: Format CE JSON spec

build: Fix Kafka client import

docs: Flesh out readme some more

feat: Simplify implementation, remove row-types
@OlaoluwaM OlaoluwaM requested a review from a team as a code owner May 12, 2025 19:59
@OlaoluwaM OlaoluwaM changed the title feat: Initialize project and add documentation feat: Initialize project (again) May 12, 2025
@pbrisbin pbrisbin removed their request for review May 12, 2025 20:15
mjgpy3
mjgpy3 previously approved these changes May 13, 2025
Copy link

@mjgpy3 mjgpy3 left a comment

Choose a reason for hiding this comment

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

LGTM. Really nice documentation throughout.

I could see us eventually splitting out different bindings into different packages (similarly how you'll see persistent for engine-agnostic stuff and persistent-postgresql for postgres-specific stuff).

@OlaoluwaM
Copy link
Contributor Author

@mjgpy3 that makes sense, I kinda just followed the structure in the JS SDK but I don't see why we couldn't split this out as you say

@mjgpy3
Copy link

mjgpy3 commented May 13, 2025

@OlaoluwaM yeah and it's really not necessary to split at the moment.

mjgpy3
mjgpy3 previously approved these changes May 13, 2025
@OlaoluwaM OlaoluwaM enabled auto-merge (squash) May 13, 2025 17:19
@OlaoluwaM OlaoluwaM requested a review from chris-martin May 13, 2025 17:25
@OlaoluwaM OlaoluwaM merged commit 18e693a into main May 13, 2025
8 checks passed
@OlaoluwaM OlaoluwaM deleted the ola/project-init-new branch May 13, 2025 17:39
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.

4 participants