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

Test server.rb with better isolation #585

Merged
merged 4 commits into from
Mar 10, 2025

Conversation

andyw8
Copy link
Contributor

@andyw8 andyw8 commented Mar 3, 2025

Since server.rb runs within the host Rails application, we should ensure we don't accidentally introduce something which depends on sorbet-runtime.

@andyw8 andyw8 requested a review from a team as a code owner March 3, 2025 22:05
@andyw8 andyw8 added the chore Chore task label Mar 3, 2025
@andyw8 andyw8 force-pushed the andyw8/test-server-with-better-isolation branch from 660d94d to 76fe8ad Compare March 3, 2025 22:27
@andyw8
Copy link
Contributor Author

andyw8 commented Mar 4, 2025

(test failures against Rails main, see #586)

@andyw8 andyw8 force-pushed the andyw8/test-server-with-better-isolation branch from 3bd7bae to aa08c1e Compare March 4, 2025 20:50
@andyw8 andyw8 enabled auto-merge (squash) March 5, 2025 15:02
@andyw8 andyw8 force-pushed the andyw8/test-server-with-better-isolation branch from aa08c1e to 4aa87d7 Compare March 7, 2025 21:19
@andyw8 andyw8 merged commit 2b25b2d into main Mar 10, 2025
28 of 29 checks passed
@andyw8 andyw8 deleted the andyw8/test-server-with-better-isolation branch March 10, 2025 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Chore task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants