Skip to content

Add page whitelist instructions to README for faster docs app dev builds#6313

Merged
carlosabadia merged 1 commit intomainfrom
carlos/add-whitelist-to-readme
Apr 9, 2026
Merged

Add page whitelist instructions to README for faster docs app dev builds#6313
carlosabadia merged 1 commit intomainfrom
carlos/add-whitelist-to-readme

Conversation

@carlosabadia
Copy link
Copy Markdown
Contributor

No description provided.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 9, 2026

Greptile Summary

This PR adds a "Page Whitelist (Faster Dev Builds)" section to docs/app/README.md, documenting how contributors can limit compiled pages by editing reflex_docs/whitelist.py. The documented rules (forward-slash prefix, no trailing slash, empty list builds all, prefix matching) are all accurate against the actual whitelist.py implementation.

Confidence Score: 5/5

Safe to merge — documentation-only change with no code modifications.

All documented rules and behaviors were verified against the actual whitelist.py implementation and are accurate. No code is changed and no P1/P0 findings exist.

No files require special attention.

Vulnerabilities

No security concerns identified.

Important Files Changed

Filename Overview
docs/app/README.md Adds a "Page Whitelist (Faster Dev Builds)" section that accurately documents the whitelist.py feature, with correct rules and examples matching the actual implementation.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Dev server starts] --> B{WHITELISTED_PAGES empty?}
    B -- Yes --> C[Build all pages]
    B -- No --> D{path == '/'}
    D -- Yes --> E[Always build root]
    D -- No --> F{Only entry is '/'}
    F -- Yes --> G[Skip page]
    F -- No --> H{path.startswith whitelisted_path?}
    H -- Yes --> I[Build page]
    H -- No --> G
Loading

Reviews (1): Last reviewed commit: "Add page whitelist instructions to READM..." | Re-trigger Greptile

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Apr 9, 2026

Merging this PR will not alter performance

✅ 9 untouched benchmarks


Comparing carlos/add-whitelist-to-readme (dd7b112) with main (08ee62a)

Open in CodSpeed

@carlosabadia carlosabadia merged commit 90cc0d9 into main Apr 9, 2026
6 checks passed
@carlosabadia carlosabadia deleted the carlos/add-whitelist-to-readme branch April 9, 2026 17:22
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.

2 participants