Skip to content

Update links in module API docs (generated from module.c)#433

Merged
zuiderkwast merged 1 commit intovalkey-io:unstablefrom
zuiderkwast:fix-module-api-links
May 3, 2024
Merged

Update links in module API docs (generated from module.c)#433
zuiderkwast merged 1 commit intovalkey-io:unstablefrom
zuiderkwast:fix-module-api-links

Conversation

@zuiderkwast
Copy link
Copy Markdown
Contributor

These are used in the docs and on the website.

Signed-off-by: Viktor Söderqvist <viktor.soderqvist@est.tech>
Copy link
Copy Markdown
Contributor Author

@zuiderkwast zuiderkwast left a comment

Choose a reason for hiding this comment

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

This is comments used for generated docs. I'm merging.

@zuiderkwast zuiderkwast added the rebranding Valkey is not Redis label May 3, 2024

# Populate the 'since' map (name => version) if we're in a git repo.
require File.dirname(__FILE__) ++ '/module-api-since.rb'
require "./" ++ File.dirname(__FILE__) ++ '/module-api-since.rb'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why was this changed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I don't know ruby that well, but with this change I can call the script regardless of my current directory.

Copy link
Copy Markdown
Contributor Author

@zuiderkwast zuiderkwast May 3, 2024

Choose a reason for hiding this comment

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

File.dirname(__FILE__) is apparently a relative path.

Copy link
Copy Markdown
Member

@madolson madolson May 3, 2024

Choose a reason for hiding this comment

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

Ok, I suppose in fullness of time it should be called from automation.

@zuiderkwast zuiderkwast merged commit 472c1ca into valkey-io:unstable May 3, 2024
@zuiderkwast zuiderkwast deleted the fix-module-api-links branch May 3, 2024 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rebranding Valkey is not Redis

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants