Skip to content

Conversation

@rajatjindal
Copy link
Member

built on top of #18

@rajatjindal rajatjindal mentioned this pull request Apr 3, 2025
4 tasks
Copy link
Contributor

@adamreese adamreese left a comment

Choose a reason for hiding this comment

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

Will need a rebase after #18 is merged

if resp.Body == nil {
t.Fatal("body is nil")
}
t.Log(resp.Status)
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this for debugging only?

Copy link
Member Author

Choose a reason for hiding this comment

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

yeah, i guess we can keep it there? happy to remove if you insist.

@rajatjindal
Copy link
Member Author

Hi @adamreese i think this branch is ready for review. thank you. (I have started work on the changes for support to redis).

@rajatjindal
Copy link
Member Author

❤️ thanks Adam.

@rajatjindal rajatjindal merged commit 8ffe6d3 into spinframework:wasip2 Apr 22, 2025
2 checks passed
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