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

Update README.md of GO and C# #1888

Merged
merged 1 commit into from
Jul 10, 2024
Merged

Conversation

GilboaAWS
Copy link
Collaborator

No description provided.

@GilboaAWS GilboaAWS force-pushed the update_readme_files branch 3 times, most recently from 479e376 to 3e21f51 Compare July 9, 2024 09:54
@barshaul
Copy link
Collaborator

barshaul commented Jul 9, 2024

Rebase after #1892 is merged so all tests will pass

@GilboaAWS GilboaAWS changed the title Edited README files for non GA status langs. Update README.md of GO and C# Jul 9, 2024
@GilboaAWS GilboaAWS requested a review from a team as a code owner July 9, 2024 14:26
go/README.md Outdated

## Current Status

We've made Valkey GLIDE an open-source project, and are releasing it in Preview to the community to gather feedback, and actively collaborate on the project roadmap. We welcome questions and contributions from all Redis stakeholders.
Copy link
Collaborator

@barshaul barshaul Jul 9, 2024

Choose a reason for hiding this comment

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

we're not releasing it as a preview. This readme should simply say that the status is under development. You can leave only the first section :

GO wrapper

The GO wrapper is currently not in a usable state and is under development.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It refers to all the Valkey Glide, however, I'll remove this section fro this file.

@ikolomi ikolomi merged commit 5fadbc4 into valkey-io:main Jul 10, 2024
10 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.

6 participants