Skip to content

Conversation

@benjaminstrasser
Copy link
Member

No description provided.

Copy link
Collaborator

@sjaghori sjaghori left a comment

Choose a reason for hiding this comment

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

we need eslint rules ('import/no-restricted-paths' package e.g.) to keep things separat:

TODO's:

  • prevents imports from src/* into services/*
  • prevents imports from src/* into shared/*
  • prevents imports from services/* into shared/*

vite.config.ts Outdated
include: ['src/**/*.{test,spec}.{js,ts}', 'services/**/*.{test,spec}.{js,ts}']
},
server: { port: 3000, fs: { allow: ['services/src/util/slug'] } }, // Add shared workspace and move slug to shared
server: { port: 3000, fs: { allow: ['shared/src'] } },
Copy link
Collaborator

Choose a reason for hiding this comment

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

a general question, do we want to have a src folder inside shared workspace? Just saying that we don't have to.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it we could get rid of the src folder.

@benjaminstrasser benjaminstrasser force-pushed the feat/project-slug branch 2 times, most recently from 0510a72 to e2b0434 Compare August 2, 2024 12:37
@benjaminstrasser benjaminstrasser force-pushed the shared-folder branch 3 times, most recently from e6618a8 to a3ce1f4 Compare August 20, 2024 12:01
Base automatically changed from feat/project-slug to main August 20, 2024 12:18
benjaminstrasser and others added 2 commits October 6, 2024 14:35
Signed-off-by: Benjamin Strasser <bp.strasser@gmail.com>
…s not supported by pnpm, fixes ci, updates pnpm, vitest & vite
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.

3 participants