Skip to content
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

Docs: fix missing separator in setup.md #764

Merged
merged 1 commit into from
Mar 8, 2025

Conversation

peter50216
Copy link
Contributor

The "Initialize Pagy" part is currently squished together with the "Builder" menu on the doc website.

The "Initialize Pagy" part is currently squished together with the "Builder" menu on the doc website.
@benkoshy
Copy link
Collaborator

benkoshy commented Mar 7, 2025

@peter50216 Brilliantly spotted.

Even after you pointed it out, it took a full 10 minutes to work out what was missing.

I'll test it out locally and get back to you.

@benkoshy
Copy link
Collaborator

benkoshy commented Mar 8, 2025

looks great

LGTM! @ddnexus This is ready for merging.

@benkoshy benkoshy changed the title Fix missing separator in setup.md Docs: fix missing separator in setup.md Mar 8, 2025
@ddnexus ddnexus merged commit 9d98aa6 into ddnexus:master Mar 8, 2025
4 checks passed
@ddnexus
Copy link
Owner

ddnexus commented Mar 8, 2025

Thank you! @peter50216 @benkoshy

@peter50216 peter50216 deleted the patch-1 branch March 8, 2025 06:58
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.

3 participants