Skip to content

test: Rollup integration tests#897

Open
timfish wants to merge 14 commits intomainfrom
timfish/test/integration-rollup
Open

test: Rollup integration tests#897
timfish wants to merge 14 commits intomainfrom
timfish/test/integration-rollup

Conversation

@timfish
Copy link
Collaborator

@timfish timfish commented Mar 6, 2026

This PR:

  • Moves the Sentry configs to separate reusable files
  • Adds tests for Rollup v3 + v4
    • These use JavaScript config files rather than TypeScript because Rollup only supports TypeScript on the latest Node versions

@github-actions
Copy link

github-actions bot commented Mar 6, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • Use crypto.randomUUID rather than uuid by timfish in #892

Internal Changes 🔧

  • Rollup integration tests by timfish in #897
  • New integration tests by timfish in #896
  • Remove lerna by timfish in #895
  • Migrate to Vitest by timfish in #894

🤖 This preview updates automatically when you update the PR.

@timfish timfish force-pushed the timfish/test/integration-rollup branch from 7613d7b to 3a43d06 Compare March 6, 2026 15:09
@@ -0,0 +1 @@
* text=auto eol=lf No newline at end of file
Copy link
Collaborator Author

@timfish timfish Mar 6, 2026

Choose a reason for hiding this comment

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

This was key to fixing the Windows line endings snapshot issues.

Without this, git on Windows by default checks out all files with modified line endings!

@timfish timfish marked this pull request as ready for review March 6, 2026 15:33
@timfish timfish requested a review from andreiborza March 6, 2026 16:12
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.

1 participant