Skip to content

Readme renewal - #216

Merged
smclacke merged 17 commits into
mainfrom
readme-renewal
Apr 28, 2026
Merged

Readme renewal#216
smclacke merged 17 commits into
mainfrom
readme-renewal

Conversation

@smclacke

@smclacke smclacke commented Apr 21, 2026

Copy link
Copy Markdown
Contributor
  • added extra link to licensing information for cheerpx.io/licensing
  • decided to leave the rest of the licensing section as is
  • cleaned up section about Claude AI, removing verbose screenshots
  • added Feature header and fixed TOC Feature header link
  • removed REPL diff block, created simple tip section and removed link from TOC

** this was branched from Merlin's 'readme-restructure' branch, which he will leave a comment about, giving details about the changes he has made.

Original branch changes

(hi Merlin here --v)
This PR and branch was build of off the documentations made prior by me in the branch readme_restructure. Changes made were:

  • Restructured and organized the different sections around user intent (quick start ->networking etc.) The original readme topics bled into each other, this separation should add clarity.

    • Broke down Networking into clear subsections
  • Reworded and restyled certain sections for added clarity (ie: merging the delay and connection status dot colour bullet points check into 1 concise tip)

  • Reworked local setup to a cleaner local serving flow.

    • Already included a custom-images directory and location block to save the user on setup steps. As well as adding clearer inline comments

@codingfrog27

codingfrog27 commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Looks good! The Claude AI section seems improved now! I've left comments on specific lines with changes. Besides these I'll lay out the further steps:

  • Fix image issue:

  • See if we can get mini image to work with sudo in local hosting. If not upload large image

  • test online deployment image again

  • test and if needed and possible fix mini image dockerfile (but if this would take too much time we leave it be)

  • Merge Alpine repo into this one

  • Bundle tips into connection tips section

@smclacke

Copy link
Copy Markdown
Contributor Author

Great, thanks @codingfrog27. I'll clean up using the comments you have made. I think it best to go ahead and merge this PR, I'll open a new one for the image + networking related updates so the PRs stay small and organised.

@codingfrog27 codingfrog27 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.

Looks good! The Claude AI section seems improved now! I've left comments on specific lines with changes. Besides these I'll lay out the further steps:

  • Fix image issue:
  • See if we can get mini image to work with sudo in local hosting. If not upload large image
  • test online deployment image again
  • test and if needed and possible fix mini image dockerfile (but if this would take too much time we leave it be)
  • Merge Alpine repo into this one
  • Bundle tips into connection tips section

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md
Comment thread README.md
Comment thread README.md Outdated
Comment thread README.md Outdated
smclacke and others added 3 commits April 28, 2026 16:14
Co-authored-by: Merlin de Cloe <89702955+codingfrog27@users.noreply.github.com>
@smclacke
smclacke merged commit 8d9a89a into main Apr 28, 2026
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