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.

Sign up for GitHub

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

Jump to bottom

Removing Redis from internal lua function names and comments #288

Open
wants to merge 3 commits into
base: unstable
Choose a base branch
from
Open

Removing Redis from internal lua function names and comments #288

wants to merge 3 commits into from

Commits on Apr 11, 2024

  1. Fixed a bug in lua debugger for 'server *' invocations and Removing R… 

    …edis from internal function names.

* Fixed a bug where 'server *' invocation from lua debugger was not working.
* Functions in lua c files lack of documentation. Adding some documentation in this commit.
* I removed Redis from function names in eval.c and script_lua.c files. I did not touch log messages in this commit. I also did not touch references to "RedisProtocol".

Signed-off-by: Parth Patel <661497+parthpatel@users.noreply.github.com>
    @parthpatel
    parthpatel committed Apr 11, 2024
    Configuration menu
    Copy the full SHA
    3cbc1c2 View commit details
    Browse the repository at this point in the history

  2. Fixes based on PR feedback. 

    * Fixed comments to be 120 line length.
* Aliased error handler function.
* Renamed REDIS_LRAND... variable to SERVER...
* I did not rename RedisProtocol as it refers to the RESP protocol.

Signed-off-by: Parth Patel <661497+parthpatel@users.noreply.github.com>
    @parthpatel
    parthpatel committed Apr 11, 2024
    Configuration menu
    Copy the full SHA
    8fdf999 View commit details
    Browse the repository at this point in the history

Commits on Apr 16, 2024

  1. Update src/script_lua.c 

    Co-authored-by: Viktor Söderqvist <viktor.soderqvist@est.tech>
    @parthpatel @zuiderkwast
    parthpatel and zuiderkwast committed Apr 16, 2024
    Configuration menu
    Copy the full SHA
    75a938a View commit details
    Browse the repository at this point in the history