Skip to content

Rename redis_init_script file and its content#357

Merged
zuiderkwast merged 1 commit into
valkey-io:unstablefrom
hwware:update-redis-init-script
Apr 24, 2024
Merged

Rename redis_init_script file and its content#357
zuiderkwast merged 1 commit into
valkey-io:unstablefrom
hwware:update-redis-init-script

Conversation

@hwware

@hwware hwware commented Apr 23, 2024

Copy link
Copy Markdown
Contributor

Rename the init script and a related .tpl file and rename variable names inside (redis to valkey). Update variables in utils/install_server.sh.

Fixes #354

Signed-off-by: hwware <wen.hui.ware@gmail.com>
@codecov

codecov Bot commented Apr 23, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.38%. Comparing base (669f1d3) to head (14b58f0).

Additional details and impacted files
@@             Coverage Diff              @@
##           unstable     #357      +/-   ##
============================================
+ Coverage     68.33%   68.38%   +0.05%     
============================================
  Files           108      108              
  Lines         61562    61562              
============================================
+ Hits          42066    42101      +35     
+ Misses        19496    19461      -35     

see 12 files with indirect coverage changes

@hwware hwware requested a review from zuiderkwast April 23, 2024 20:51

@zuiderkwast zuiderkwast left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good.

We can mention the install script in the pr description too.

@enjoy-binbin enjoy-binbin linked an issue Apr 24, 2024 that may be closed by this pull request
@zuiderkwast zuiderkwast merged commit d09a59c into valkey-io:unstable Apr 24, 2024
@zuiderkwast zuiderkwast added the needs-doc-pr This change needs to update a documentation page. Remove label once doc PR is open. label Apr 24, 2024
hwware pushed a commit to valkey-io/valkey-doc that referenced this pull request Apr 24, 2024
Fix docs for valkey-io/valkey#357

Signed-off-by: Viktor Söderqvist <viktor.soderqvist@est.tech>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-doc-pr This change needs to update a documentation page. Remove label once doc PR is open.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rename and change variable names in utils/redis_init_script

3 participants