Skip to content

docs: migrate wiki content to wiki/ folder#48

Merged
jonathanperis merged 1 commit into
mainfrom
migrate-wiki-to-folder
May 2, 2026
Merged

docs: migrate wiki content to wiki/ folder#48
jonathanperis merged 1 commit into
mainfrom
migrate-wiki-to-folder

Conversation

@jonathanperis
Copy link
Copy Markdown
Owner

@jonathanperis jonathanperis commented May 2, 2026

Migrate GitHub wiki markdown files to wiki/ folder in main repo. Wiki will be disabled after merge.

Summary by CodeRabbit

Documentation

  • Added comprehensive project documentation covering:
    • Getting Started guide with setup instructions and API examples
    • Architecture and infrastructure specifications
    • Challenge requirements and API endpoint details
    • Performance benchmarks and resource constraints
    • CI/CD pipeline workflows and automation processes

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 2, 2026

📝 Walkthrough

Walkthrough

This PR adds six documentation files to the wiki/ directory that collectively establish a project documentation structure for a Rust-based backend challenge implementation. The files cover system architecture, challenge requirements, getting started instructions, performance metrics, and CI/CD pipeline configuration.

Changes

Wiki Documentation

Layer / File(s) Summary
Index & Navigation
wiki/index.md
Project overview page with wiki table of contents, key features list, and navigation structure.
Challenge & Architecture
wiki/Challenge.md, wiki/Architecture.md
Challenge specification describing required endpoints and resource constraints; architecture guide detailing two Actix-web API instances, Nginx load balancing, PostgreSQL with stored procedures, and observability stack.
Setup & Reference
wiki/Getting-Started.md, wiki/Performance.md
Getting started guide with prerequisites, clone/run instructions, example curl requests; performance documentation with resource constraints, actual usage metrics, and stress-testing approach.
Automation
wiki/CI-CD-Pipeline.md
CI/CD pipeline documentation describing four GitHub Actions workflows (build-check, main-release, codeql, deploy) with triggers, steps, and purposes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title 'docs: migrate wiki content to wiki/ folder' directly and accurately describes the main change—moving wiki documentation files into a wiki/ folder structure.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch migrate-wiki-to-folder

Review rate limit: 3/10 reviews remaining, refill in 41 minutes and 27 seconds.

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

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 4

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@wiki/Challenge.md`:
- Line 26: The Markdown link on Line 26 is malformed (contains "L("); update the
link syntax in Challenge.md by replacing the broken fragment
"rinha-de-backend-2024-q1L(" with a proper Markdown link such as
[github.com/zanfranceschi/rinha-de-backend-2024-q1](https://github.com/zanfranceschi/rinha-de-backend-2024-q1.md)
so the URL renders as a clickable link.

In `@wiki/CI-CD-Pipeline.md`:
- Around line 27-29: Update the CI-CD doc text to match the actual deploy.yml
behavior: replace the claim that it directly uses actions/deploy-pages and only
triggers on push with a note that the repo invokes a reusable deployment
workflow (deploy.yml) and that the workflow supports both push to main and
manual runs via workflow_dispatch; also keep the intent to publish the docs/
directory to GitHub Pages but describe that deployment is performed through the
reusable deploy.yml workflow.

In `@wiki/index.md`:
- Around line 11-13: Fix the malformed markdown links by replacing the incorrect
"L(" sequence with the proper closing bracket and paren "](" in the link labels
found (e.g., change "[Getting StartedL(Getting-Started.md)" to "[Getting
Started](Getting-Started.md)" and "[CI/CD PipelineL(CI-CD-Pipeline.md)" to
"[CI/CD Pipeline](CI-CD-Pipeline.md)"); scan the file for other occurrences of
"L(" (such as the "Performance" or any duplicated instances) and correct them so
all link text is enclosed in [] and targets in ().

In `@wiki/Performance.md`:
- Line 22: The markdown link on the line starting with "Load tests are run using
the shared
[rinha2-back-end-k6L(https://github.com/jonathanperis/rinha2-back-end-k6.md)
test suite..." is malformed; update it to use proper markdown link syntax and a
correct URL by replacing the broken token
"[rinha2-back-end-k6L(https://github.com/jonathanperis/rinha2-back-end-k6.md)"
with a well-formed link such as
"[rinha2-back-end-k6](https://github.com/jonathanperis/rinha2-back-end-k6)"
(keep the rest of the sentence unchanged), ensuring the repository name
"rinha2-back-end-k6" is the link text and the URL does not include the stray
".md" or extra characters.
🪄 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 Plus

Run ID: cc983a87-8e6f-4eee-ae5d-b3d5e42c3545

📥 Commits

Reviewing files that changed from the base of the PR and between d398c54 and 3f19b8d.

📒 Files selected for processing (6)
  • wiki/Architecture.md
  • wiki/CI-CD-Pipeline.md
  • wiki/Challenge.md
  • wiki/Getting-Started.md
  • wiki/Performance.md
  • wiki/index.md

Comment thread wiki/Challenge.md

## Source

Full specification: [github.com/zanfranceschi/rinha-de-backend-2024-q1L(https://github.com/zanfranceschi/rinha-de-backend-2024-q1.md)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Fix broken source link syntax on Line 26.

The link is malformed (L(), so it won’t render as clickable Markdown.

Suggested fix
-Full specification: [github.com/zanfranceschi/rinha-de-backend-2024-q1L(https://github.com/zanfranceschi/rinha-de-backend-2024-q1.md)
+Full specification: [github.com/zanfranceschi/rinha-de-backend-2024-q1](https://github.com/zanfranceschi/rinha-de-backend-2024-q1)
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
Full specification: [github.com/zanfranceschi/rinha-de-backend-2024-q1L(https://github.com/zanfranceschi/rinha-de-backend-2024-q1.md)
Full specification: [github.com/zanfranceschi/rinha-de-backend-2024-q1](https://github.com/zanfranceschi/rinha-de-backend-2024-q1)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@wiki/Challenge.md` at line 26, The Markdown link on Line 26 is malformed
(contains "L("); update the link syntax in Challenge.md by replacing the broken
fragment "rinha-de-backend-2024-q1L(" with a proper Markdown link such as
[github.com/zanfranceschi/rinha-de-backend-2024-q1](https://github.com/zanfranceschi/rinha-de-backend-2024-q1.md)
so the URL renders as a clickable link.

Comment thread wiki/CI-CD-Pipeline.md
Comment on lines +27 to +29
- **Trigger:** Push to main branch
- **Steps:** Deploys the `docs/` directory to GitHub Pages using the actions/deploy-pages workflow
- **Purpose:** Publish project documentation and stress test reports to GitHub Pages
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Align deploy workflow description with actual deploy.yml.

Current text says this repo directly uses actions/deploy-pages and only triggers on push, but .github/workflows/deploy.yml uses a reusable workflow and also supports workflow_dispatch.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@wiki/CI-CD-Pipeline.md` around lines 27 - 29, Update the CI-CD doc text to
match the actual deploy.yml behavior: replace the claim that it directly uses
actions/deploy-pages and only triggers on push with a note that the repo invokes
a reusable deployment workflow (deploy.yml) and that the workflow supports both
push to main and manual runs via workflow_dispatch; also keep the intent to
publish the docs/ directory to GitHub Pages but describe that deployment is
performed through the reusable deploy.yml workflow.

Comment thread wiki/index.md
Comment on lines +11 to +13
| [Getting StartedL(Getting-Started.md) | Prerequisites and how to run |
| [Performance](Performance) | Results, benchmarks, resource usage |
| [CI/CD PipelineL(CI-CD-Pipeline.md) | GitHub Actions workflows |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Fix malformed links in the wiki index (navigation is currently broken).

Several links use L( instead of ](, so they won’t render correctly.

Suggested fix
-| [Getting StartedL(Getting-Started.md) | Prerequisites and how to run |
+| [Getting Started](Getting-Started.md) | Prerequisites and how to run |
@@
-| [CI/CD PipelineL(CI-CD-Pipeline.md) | GitHub Actions workflows |
+| [CI/CD Pipeline](CI-CD-Pipeline.md) | GitHub Actions workflows |
@@
-*[GitHubL(https://github.com/jonathanperis/rinha2-back-end-rust.md) · [Jonathan Peris](https://jonathanperis.github.io/)*
+*[GitHub](https://github.com/jonathanperis/rinha2-back-end-rust) · [Jonathan Peris](https://jonathanperis.github.io/)*

Also applies to: 24-24

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@wiki/index.md` around lines 11 - 13, Fix the malformed markdown links by
replacing the incorrect "L(" sequence with the proper closing bracket and paren
"](" in the link labels found (e.g., change "[Getting
StartedL(Getting-Started.md)" to "[Getting Started](Getting-Started.md)" and
"[CI/CD PipelineL(CI-CD-Pipeline.md)" to "[CI/CD Pipeline](CI-CD-Pipeline.md)");
scan the file for other occurrences of "L(" (such as the "Performance" or any
duplicated instances) and correct them so all link text is enclosed in [] and
targets in ().

Comment thread wiki/Performance.md

## Stress Testing

Load tests are run using the shared [rinha2-back-end-k6L(https://github.com/jonathanperis/rinha2-back-end-k6.md) test suite, which simulates concurrent users performing debits, credits, validations, and statement queries.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Fix broken stress-test suite link on Line 22.

The link is malformed and currently won’t render correctly.

Suggested fix
-Load tests are run using the shared [rinha2-back-end-k6L(https://github.com/jonathanperis/rinha2-back-end-k6.md) test suite, which simulates concurrent users performing debits, credits, validations, and statement queries.
+Load tests are run using the shared [rinha2-back-end-k6](https://github.com/jonathanperis/rinha2-back-end-k6) test suite, which simulates concurrent users performing debits, credits, validations, and statement queries.
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
Load tests are run using the shared [rinha2-back-end-k6L(https://github.com/jonathanperis/rinha2-back-end-k6.md) test suite, which simulates concurrent users performing debits, credits, validations, and statement queries.
Load tests are run using the shared [rinha2-back-end-k6](https://github.com/jonathanperis/rinha2-back-end-k6) test suite, which simulates concurrent users performing debits, credits, validations, and statement queries.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@wiki/Performance.md` at line 22, The markdown link on the line starting with
"Load tests are run using the shared
[rinha2-back-end-k6L(https://github.com/jonathanperis/rinha2-back-end-k6.md)
test suite..." is malformed; update it to use proper markdown link syntax and a
correct URL by replacing the broken token
"[rinha2-back-end-k6L(https://github.com/jonathanperis/rinha2-back-end-k6.md)"
with a well-formed link such as
"[rinha2-back-end-k6](https://github.com/jonathanperis/rinha2-back-end-k6)"
(keep the rest of the sentence unchanged), ensuring the repository name
"rinha2-back-end-k6" is the link text and the URL does not include the stray
".md" or extra characters.

@jonathanperis jonathanperis merged commit 1a8adb5 into main May 2, 2026
5 checks passed
@jonathanperis jonathanperis deleted the migrate-wiki-to-folder branch May 2, 2026 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant