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

[Tapioca Add-on] Trigger ActiveSupport::TestCase DSL compiler on fixture changes #2163

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

andyw8
Copy link
Contributor

@andyw8 andyw8 commented Jan 22, 2025

Addresses https://github.com/Shopify/team-ruby-dx/issues/1305

TODO:

  • Point to ruby-lsp-rails once released (and update RBI)
  • Verify performance in Core

Note: This doesn't work on Core because the fixtures is compiler is disabled and Core uses a custom compiler.

@andyw8 andyw8 added the enhancement New feature or request label Jan 24, 2025
@andyw8 andyw8 force-pushed the andyw8/watch-for-yaml-changes branch from 3efd5d3 to 82e2ec7 Compare January 27, 2025 16:19
@andyw8 andyw8 marked this pull request as ready for review January 27, 2025 16:28
@andyw8 andyw8 requested a review from a team as a code owner January 27, 2025 16:28
@andyw8
Copy link
Contributor Author

andyw8 commented Jan 27, 2025

I still need to test on Core but feel free to review.

@andyw8 andyw8 force-pushed the andyw8/watch-for-yaml-changes branch from 82e2ec7 to 3ca8695 Compare January 27, 2025 16:30
@andyw8 andyw8 force-pushed the andyw8/watch-for-yaml-changes branch from 3ca8695 to 64a97f3 Compare January 27, 2025 16:59
@andyw8 andyw8 force-pushed the andyw8/watch-for-yaml-changes branch 2 times, most recently from cd353e3 to 3254f25 Compare January 27, 2025 20:32
@KaanOzkan KaanOzkan changed the title [LSP] Trigger ActiveSupport::TestCase DSL compiler on fixture changes [Tapioca Add-on] Trigger ActiveSupport::TestCase DSL compiler on fixture changes Jan 27, 2025
@andyw8 andyw8 force-pushed the andyw8/watch-for-yaml-changes branch from 9613eaf to 2521a35 Compare January 27, 2025 21:21
@andyw8 andyw8 enabled auto-merge January 27, 2025 21:21
@andyw8 andyw8 merged commit 36b4bcc into main Jan 27, 2025
33 checks passed
@andyw8 andyw8 deleted the andyw8/watch-for-yaml-changes branch January 27, 2025 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants