-
Notifications
You must be signed in to change notification settings - Fork 47
Use doc-tools CLI for GitHub Actions #1138
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
Conversation
✅ Deploy Preview for redpanda-docs-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 📝 WalkthroughWalkthroughThis set of changes removes several custom Node.js and Bash scripts used for fetching versions, generating compatibility matrices, and installing dependencies. Instead, these functionalities are replaced by invoking equivalent commands from the Sequence Diagram(s)sequenceDiagram
participant Workflow
participant npx doc-tools
participant GitHub API
participant Local System
Workflow->>npx doc-tools: Run fetch/get/install command
npx doc-tools->>GitHub API: Fetch required data (versions/files)
GitHub API-->>npx doc-tools: Return data
npx doc-tools->>Local System: Save or process data
npx doc-tools-->>Workflow: Command complete
Suggested reviewers
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Description
Review deadline: 29 May
This pull request simplifies the workflows by replacing a custom script with a standardized tool (
doc-tools
) and removes the associated custom code. It also updates paths for generating documentation. Below are the key changes:Workflow Updates
fetch.js
withnpx doc-tools fetch
in thefetch-acceptance-tests.yml
andget-cloud-api-spec.yml
workflows for fetching files, simplifying the dependency management and execution process. [1] [2]generate-crd.yml
workflow to reflect the new directory structure.Codebase Cleanup
fetch.js
script and its dependencies (@octokit/rest
,chalk
, etc.) by deletingscripts/fetch-from-github/fetch.js
,package.json
, andpackage-lock.json
, as they are no longer needed with the adoption ofdoc-tools
. [1] [2] [3]Page previews
Checks