Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to consider lib/**/server/** as server files #12732

Open
rChaoz opened this issue Sep 28, 2024 · 1 comment · May be fixed by #12733
Open

Add option to consider lib/**/server/** as server files #12732

rChaoz opened this issue Sep 28, 2024 · 1 comment · May be fixed by #12733
Labels
feature request New feature or request

Comments

@rChaoz
Copy link

rChaoz commented Sep 28, 2024

Describe the problem

Sometimes client-side and server-side code is tightly coupled, and it make senses to keep it together, but I don't want a bunch of abc.server.ts files. Instead, I'd like to group all serverside files into a directory.

Describe the proposed solution

Add an option to consider consider lib/**/server/** files as server-side, instead of lib/server/**. This would allow for folder structures like:

lib/
  services/
    email/
      model.ts
      client.ts
      server/
        client.ts
        api.ts
        config.ts
    content/
      urls.ts
      model.ts
      upload_request.ts
      server/
        upload.ts
        manage.ts

Alternatives considered

Just name all serverside files with .server in their name.

Importance

would make my life easier

@rChaoz rChaoz linked a pull request Sep 28, 2024 that will close this issue
6 tasks
@rChaoz rChaoz changed the title Consider lib/**/server/** as server files Add option to consider lib/**/server/** as server files Sep 28, 2024
@Kapsonfire-DE
Copy link
Contributor

Kapsonfire-DE commented Oct 8, 2024

#12477

#12529

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants