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

Add some missing INFO fields about FUNCTION #2599

Merged
merged 3 commits into from
Dec 31, 2023

Conversation

enjoy-binbin
Copy link
Collaborator

@enjoy-binbin enjoy-binbin commented Nov 9, 2023

Info fields from the rearrangement of scripts when functions were introduced.
see redis/redis#9780

Copy link

netlify bot commented Nov 9, 2023

👷 Deploy request for redis-doc pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 2f6802c

@enjoy-binbin
Copy link
Collaborator Author

@MeirShpilraien i'm not particularly familiar with it, so please give it a look when you have times, thanks

@oranagra oranagra added the to-be-merged should probably be merged soon label Dec 27, 2023
@oranagra
Copy link
Member

made some adjustments from memory, hope i got them right.

@enjoy-binbin
Copy link
Collaborator Author

made some adjustments from memory, hope i got them right.

it is good, thanks for the help, let's merge it

@oranagra oranagra merged commit 98dd384 into redis:master Dec 31, 2023
1 check failed
@enjoy-binbin enjoy-binbin deleted the add_function_infos branch January 2, 2024 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
to-be-merged should probably be merged soon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants