Skip to content

Conversation

@mixmix
Copy link

@mixmix mixmix commented Mar 20, 2025

I refactored this code on a call with @nanomonkey - we're using it to try and write a spec, and found it very hard to read in it's native state. This PR is opened so that

  • we have a clearer reference for spec writing
  • we have an example of what more accessible code looks like (IMO)
    • comments
    • grouping of code into more coherent chunks
      • collecting related code
      • putting white space in to aid reading

🔥 TODO before merge:

  • write tests over this function (check they pass before + after)
  • resolve duplication across simplechat + simplepub folders 😭

@mixmix mixmix marked this pull request as draft March 20, 2025 22:36
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