Skip to content

Add minimum and recommended system requirements to local setup guide (#553) - #576

Open
ruchiralakshan123 wants to merge 4 commits into
wso2:5.1.xfrom
ruchiralakshan123:add-system-requirements
Open

Add minimum and recommended system requirements to local setup guide (#553)#576
ruchiralakshan123 wants to merge 4 commits into
wso2:5.1.xfrom
ruchiralakshan123:add-system-requirements

Conversation

@ruchiralakshan123

@ruchiralakshan123 ruchiralakshan123 commented Jul 12, 2026

Copy link
Copy Markdown

Purpose

Resolves #553

Goals

Specify the minimum and recommended system requirements in the local setup documentation to help users validate their environments before installation.

Approach

Added a new "System requirements" section with a comparison table (detailing OS, CPU, RAM, Disk Space, and Network specifications) to en/docs/get-started/setup/local-setup.md.

User stories

N/A

Release note

Add minimum and recommended system requirements to local setup guide.

Documentation

N/A (This is a documentation-only fix)

Training

N/A

Certification

N/A

Marketing

N/A

Automation tests

  • Unit tests
    N/A
  • Integration tests
    N/A

Security checks

Samples

N/A

Related PRs

N/A

Migrations (if applicable)

N/A

Test environment

Verified markdown formatting locally.

Learning

N/A

Summary by CodeRabbit

  • Documentation
    • Added a new System requirements section to the local setup guide.
    • Documented minimum and recommended operating system, CPU, memory, disk space, and network requirements.

@coderabbitai

coderabbitai Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The local setup documentation adds a “System requirements” section with minimum and recommended requirements for operating system, CPU, RAM, disk space, and network access.

Changes

Local setup requirements

Layer / File(s) Summary
Document system requirements
en/docs/get-started/setup/local-setup.md
Adds a minimum-versus-recommended requirements table covering operating system, CPU, RAM, disk space, and network access.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: anupama-pathirage, kavinduzoysa, nipunaranasinghe, rosensilva

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The added system requirements section covers the requested minimum/recommended OS, CPU, RAM, disk space, and network prerequisites.
Out of Scope Changes check ✅ Passed The PR is limited to the local setup documentation and does not introduce unrelated changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title clearly and concisely identifies the addition of minimum and recommended system requirements to the local setup guide.
Description check ✅ Passed The description covers the required sections and explains the documentation change, its purpose, approach, release note, testing, and impact. The Documentation section could link directly to the updat…
Full details: Description check

Explanation

The description covers the required sections and explains the documentation change, its purpose, approach, release note, testing, and impact. The Documentation section could link directly to the updated page instead of using N/A, but the description is otherwise complete.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@en/docs/get-started/setup/local-setup.md`:
- Line 21: Update the Network requirement in the local setup prerequisites to
require internet only for installation and dependency downloads; remove WSO2
Cloud connectivity as a minimum prerequisite while preserving the existing
broadband recommendation.
- Line 17: Update the operating-system support row in the local setup
requirements table to include Fedora alongside the existing Linux distributions,
matching the RHEL/Fedora RPM installation instructions. Keep the macOS and
Windows requirements unchanged.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d1b18288-feeb-4050-8146-f35ec6566adc

📥 Commits

Reviewing files that changed from the base of the PR and between 7e6d503 and d4241af.

📒 Files selected for processing (1)
  • en/docs/get-started/setup/local-setup.md

Comment thread en/docs/get-started/setup/local-setup.md Outdated
Comment thread en/docs/get-started/setup/local-setup.md Outdated

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@en/docs/get-started/setup/local-setup.md`:
- Line 20: Add blank lines immediately before and after the requirements table
in the setup documentation, preserving the table content and formatting while
satisfying Markdownlint MD058.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5d0626af-5802-4bc8-89e9-3079b3ad198f

📥 Commits

Reviewing files that changed from the base of the PR and between d4241af and 632fd0d.

📒 Files selected for processing (1)
  • en/docs/get-started/setup/local-setup.md

Comment thread en/docs/get-started/setup/local-setup.md
@AmilaSamith AmilaSamith added the Contributor Assigned to pull requests received from external contributors label Sep 1, 2026
@AmilaSamith
AmilaSamith requested a lite review from Copilot September 1, 2026 10:11

Copilot AI 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.

🟡 Changes recommended

The documentation now contains a duplicated “Installation steps” heading that should be removed to avoid confusing rendered output and duplicate anchors.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Adds system requirement guidance to the Local Setup documentation so users can validate their environment before installing WSO2 Integrator IDE.

Changes:

  • Introduced a new System requirements section with a minimum vs. recommended comparison table.
  • Kept the existing installation flow, now preceded by the requirements section.
File summaries
File Description
en/docs/get-started/setup/local-setup.md Adds a system requirements section/table ahead of the installation steps.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread en/docs/get-started/setup/local-setup.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@AmilaSamith
AmilaSamith changed the base branch from main to 5.1.x September 2, 2026 03:56
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

Broken links, images & orphan pages

Passing — no broken links or images found.

Links/images come from one crawl of the production build (baseUrl-aware). Orphans are docs not referenced by sidebars.ts.

Summary

  • Broken links & images — total 0 · 🆕 introduced 0 · 📄 already on main 0
  • Orphan pages — total 8 · 🆕 introduced 0 · 📄 already on main 8

Broken links & images

Introduced by this PR

No new broken link(s)/image(s) introduced by this PR. ✅

Already on main — 0 total

None.

Orphan pages

Introduced by this PR

No new orphan page(s) introduced by this PR. ✅

Already on main — 8 total

Already present on the base branch (not caused by this PR):

Show 8
  • docs/deploy-operate/observe/datadog-integration
  • docs/deploy-operate/observe/elastic-stack-elk
  • docs/deploy-operate/observe/metrics-prometheus-grafana
  • docs/deploy-operate/observe/opensearch-integration
  • docs/deploy-operate/observe/recipe-elk-stack
  • docs/deploy-operate/observe/recipe-kubernetes-production
  • docs/deploy-operate/observe/recipe-local-development
  • docs/deploy-operate/observe/recipe-opensearch-setup

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Contributor Assigned to pull requests received from external contributors

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Minimum and recommended system requirements are missing in the Local Setup documentation

3 participants