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

fix: separate vite and vitest configs #193

Merged
merged 1 commit into from
Jun 21, 2023

Conversation

diivi
Copy link
Contributor

@diivi diivi commented Jun 21, 2023

Fix for https://discordapp.com/channels/714698561081704529/928693344358514698/1121122996384432148, an issue where the build had not manifest.json because of incorrect vite config.

This PR updates the .eslintrc.js file to include vitest.config.ts in the ignorePatterns array. It also updates the vite.config.ts file to import defineConfig from vite instead of vitest/config. Additionally, it removes the test object from the vite.config.ts file and creates a new vitest.config.ts file to configure the testing environment.

Generated using OpenSauced.

@diivi
Copy link
Contributor Author

diivi commented Jun 21, 2023

@bdougie please check for that issue on this branch once, I was able to generate a manifest in the build with this.

@bdougie bdougie merged commit 7a980c8 into beta Jun 21, 2023
github-actions bot pushed a commit that referenced this pull request Jun 21, 2023
## [1.9.0-beta.2](v1.9.0-beta.1...v1.9.0-beta.2) (2023-06-21)

### 🐛 Bug Fixes

* separate vite and vitest configs ([#193](#193)) ([7a980c8](7a980c8))
@github-actions
Copy link

🎉 This PR is included in version 1.9.0-beta.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this pull request Jun 29, 2023
## [1.9.0](v1.8.0...v1.9.0) (2023-06-29)

### ✅ Tests

* Tests for colorPreference and getUserPRVelocity ([#190](#190)) ([6b9a5c3](6b9a5c3))

### 🍕 Features

* post to linkedin projects ([#177](#177)) ([a20bff4](a20bff4))

### 🐛 Bug Fixes

* Made the "Tools:" conditional according to the current page ([#197](#197)) ([b08ad07](b08ad07)), closes [issue#112](https://github.com/open-sauced/issue/issues/112) [issue#112](https://github.com/open-sauced/issue/issues/112)
* separate vite and vitest configs ([#193](#193)) ([7a980c8](7a980c8))
@github-actions
Copy link

🎉 This PR is included in version 1.9.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

zer0and1 pushed a commit to zer0and1/open-sauced.ai that referenced this pull request Jul 26, 2023
## [1.9.0-beta.2](open-sauced/ai@v1.9.0-beta.1...v1.9.0-beta.2) (2023-06-21)

### 🐛 Bug Fixes

* separate vite and vitest configs ([#193](open-sauced/ai#193)) ([7a980c8](open-sauced/ai@7a980c8))
zer0and1 pushed a commit to zer0and1/open-sauced.ai that referenced this pull request Jul 26, 2023
## [1.9.0](open-sauced/ai@v1.8.0...v1.9.0) (2023-06-29)

### ✅ Tests

* Tests for colorPreference and getUserPRVelocity ([#190](open-sauced/ai#190)) ([6b9a5c3](open-sauced/ai@6b9a5c3))

### 🍕 Features

* post to linkedin projects ([#177](open-sauced/ai#177)) ([a20bff4](open-sauced/ai@a20bff4))

### 🐛 Bug Fixes

* Made the "Tools:" conditional according to the current page ([#197](open-sauced/ai#197)) ([b08ad07](open-sauced/ai@b08ad07)), closes [issue#112](https://github.com/open-sauced/issue/issues/112) [issue#112](https://github.com/open-sauced/issue/issues/112)
* separate vite and vitest configs ([#193](open-sauced/ai#193)) ([7a980c8](open-sauced/ai@7a980c8))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants