Details
Diff Detail
- Repository
- rCOMM Comm
- Branch
- jonringer/nix-dev-docs (branched from master)
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
docs/nix_shared_workflows.md | ||
---|---|---|
13–15 ↗ | (On Diff #16035) | Do we have to include info about Host (localhost), Password (the one you made when initializing the MariaDB server in the previous step), or Port (3306 by default)? |
14 ↗ | (On Diff #16035) | |
14 ↗ | (On Diff #16035) | |
15 ↗ | (On Diff #16035) | Match the formatting used on the other lines (values in parentheses). |
15 ↗ | (On Diff #16035) | Should this be "your username" or something similar? Is it clear what "your user" means to a user or can this be clearer? |
Address comments
docs/nix_shared_workflows.md | ||
---|---|---|
13–15 ↗ | (On Diff #16035) | For tableplus, no |
15 ↗ | (On Diff #16035) | This is distinct from the prompt you see when you log into the computer. I could also ask the user to do echo $MYSQL_UNIX_PORT which will also contain this port. |
15 ↗ | (On Diff #16035) |
I'm not asking them for the user though, the socket path needs to look like /Users/jon/.local/share/MariaDB/mysql.sock. Going to ask for socket path |
almost done!
docs/nix_shared_workflows.md | ||
---|---|---|
14–15 |
Ok got it. What about something like this then? Since it's on the same line it's clearer that they're related (the socket path needs to be entered along with checking Use socket) Also pedantic, but unsure if we should do “Use socket”. vs “Use socket.” I think the correct thing to do is the latter, but it's up to you. | |
13–15 ↗ | (On Diff #16035) | Wait, really? I'm asking because in dev_environment.md we include all those details, and shouldn't the setup be the same? Why wouldn't it be? |
docs/nix_shared_workflows.md | ||
---|---|---|
7 ↗ | (On Diff #16089) | I recognize this is copy-pasted, but I noticed two sentences in a row here that start with "After". See inline suggestion |
9 ↗ | (On Diff #16089) | I recognize that this is copy-pasted, but I think it's weird that we're using bullet points here. Can we just combine this sentence with the previous paragraph? |
13–14 ↗ | (On Diff #16089) | Previously, there was a consistent format here: - Field name (text). Now the format seems inconsistent... Maybe we should just replace Name (Comm) with a sentence? See the suggested edit |
docs/nix_shared_workflows.md | ||
---|---|---|
14–15 | “Use socket”. seems better to me. As the that's how the option is displayed. | |
13–15 ↗ | (On Diff #16035) | The unix socket is convenient because it allows you to use mariadb without having to set up a privileged user. Although I do this for this for the db_config.json anyway, it also allows for me not to have the user copy those details. I think the unix socket is a better experience for development, so I would rather use it. Both workflows work after mariadb is started with nix develop. |