Skip to content

♻️ Migrate to PNPM workspaces + Biomejs + Vitest #4

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

Merged
merged 7 commits into from
Feb 5, 2025

Conversation

alexmarqs
Copy link
Member

@alexmarqs alexmarqs commented Feb 5, 2025

This MR includes:

  • PNPM for package/workspace management
  • Biomejs for format and linting
  • Vitest for testing

I also cleaned up a few things not being used. Additionally, I removed the CI release stage as it was not being used and if we want to introduce more packages in the future we need a better way to manage releases in the monorepo (for example, using changesets - but that's something for new MR's)

(+5,202 −21,237 🤩)

Cheers.

…sting

- Switch from npm to pnpm as package manager
- Update package.json scripts to use pnpm
- Remove package-lock.json and package-lock files
- Update CI workflow to use pnpm
- Add pnpm version to package.json
- Update contributors list
- Replace Jest with Vitest for testing
- Simplify tsconfig and test configuration
- Remove unnecessary files and dependencies
- Replace ESLint and Prettier with Biome
- Add Biome configuration file
- Update package.json scripts to use Biome
- Configure VS Code settings for Biome
- Remove ESLint and related configuration files
- Update import statements and code style to match Biome recommendations
@alexmarqs alexmarqs requested a review from jpinho February 5, 2025 14:52
Copy link
Member

@jpinho jpinho left a comment

Choose a reason for hiding this comment

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

Looks great buddy! Thank you so much for taking the time, this is big ❤️

@alexmarqs
Copy link
Member Author

Looks great buddy! Thank you so much for taking the time, this is big ❤️

More coming soon! cheers!

@alexmarqs alexmarqs merged commit 2e6fe58 into epilot-dev:main Feb 5, 2025
1 check passed
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