Skip to content

Fix nx lint setup#3234

Merged
charlesBochet merged 5 commits intomainfrom
fix-nx-lint-setup
Jan 4, 2024
Merged

Fix nx lint setup#3234
charlesBochet merged 5 commits intomainfrom
fix-nx-lint-setup

Conversation

@charlesBochet
Copy link
Copy Markdown
Member

@charlesBochet charlesBochet commented Jan 4, 2024

In this PR, I'm improving nx dev experience and fixing a few bugs.

  1. Removing packages from the root package.json, they should always be in their own package
  2. Introducing a nx command in each package.json to run yarn command through nx easily: yarn nx lint / yarn nx database:reset ...
  3. Simplifying linter config: There is one .eslintrc.js at the root, and each package is free to extend it or to start from fresh one.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 4, 2024

TODOs/FIXMEs:

  • // TODO if we ever hit API Rate Limiting: packages/twenty-website/src/app/developers/contributors/api/generate/route.tsx
  • // TODO if we ever hit API Rate Limiting: packages/twenty-website/src/app/developers/contributors/api/generate/route.tsx

Generated by 🚫 dangerJS against d2cfdb7

Fixes

Fix

Fixes
@charlesBochet charlesBochet merged commit c15e138 into main Jan 4, 2024
@charlesBochet charlesBochet deleted the fix-nx-lint-setup branch January 4, 2024 15:39
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.

2 participants