Skip to content

Conversation

@deltork
Copy link
Collaborator

@deltork deltork commented Jan 16, 2026

Read me for tts-middle ware

@deltork deltork requested review from dlothian and joanise January 16, 2026 21:32
Copy link
Collaborator

@dlothian dlothian left a comment

Choose a reason for hiding this comment

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

Beautiful :')

Copy link
Member

@joanise joanise left a comment

Choose a reason for hiding this comment

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

This is very nice, thank you!

A small suggestion: when you put code blocks in a bullet list, if you indent the triple backticks along with the contents by three spaces (yes, annoyingly, 3!) it formats the code block correctly as part of the numbered or itemized list.

Compare

ImageImage

with

ImageImage

I put some code suggestions to that effect below, but they're not showing properly for me, so I suspect they won't work if you just accept them, so better redo them manually if you agree.

PS: using 7 (!) spaces also works: that's the three spaces to say you're continuing a list item plus the four spaces to say it's a code block without guards:

ImageImage

Comment on lines +30 to +32
```bash
   cd path/to/your/tts-middleware
```
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
```bash
   cd path/to/your/tts-middleware
```
```bash
   cd path/to/your/tts-middleware

Comment on lines +37 to +39
```bash
   npm install
```
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
```bash
   npm install
```
```bash
   npm install

Comment on lines +59 to +62
```bash
node index.js
```

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
```bash
node index.js
```
```bash
node index.js

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.

4 participants