masochist icon indicating copy to clipboard operation
masochist copied to clipboard

Audit {server,client,common} files

Open wincent opened this issue 8 years ago • 1 comments

Supposed to be:

  • server: only ever run on the server
  • client: only ever run on the client
  • common: run on both

But now that we have server rendering, there are a bunch of components in "client" that no longer belong there. Likewise, there is probably stuff under "server" that maybe needs to go to "common", or stuff in "common" that could be demoted to "server" or "client".

Historically, I think I also had the notion that "common" would hold stuff used by the CLI and either the "server" or "client" modules, but that distinction was not well represented with these directory names (because "bin" is not even a peer of "server", "client" and "common").

Unclear whether I should have:

  • server
  • client (will have almost nothing in it)

Or:

  • server
  • app

Or:

  • server
  • app (probably not clear what this is)
  • client

etc

wincent avatar May 12 '17 15:05 wincent

  • server
  • client
  • common
  • cli

wincent avatar Sep 30 '17 06:09 wincent

Things are much better separated in the next branch, so closing this.

wincent avatar Mar 11 '24 15:03 wincent