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

Add optional ToString() interceptor #94

Merged
merged 12 commits into from
Oct 13, 2024
Merged

Add optional ToString() interceptor #94

merged 12 commits into from
Oct 13, 2024

Conversation

andrewlock
Copy link
Owner

@andrewlock andrewlock commented Sep 30, 2024

  • Adds an optional interceptor for the ToString() method.
  • Bumps the Roslyn API to 4.11.0 as uses the new interceptor API.
    • Requires the 8.0.4xx+ SDK
  • Adds a helper for passing analyzer config to tests
  • Allow optionally enabling/disabling interceptors

@andrewlock andrewlock force-pushed the interceptors branch 4 times, most recently from 2770b11 to 3827d04 Compare October 6, 2024 15:11
Copy link

github-actions bot commented Oct 12, 2024

Test Results

   127 files  + 38     127 suites  +38   3m 42s ⏱️ +21s
 3 909 tests +  8   3 909 ✅ +  8  0 💤 ±0  0 ❌ ±0 
72 774 runs  +116  72 774 ✅ +116  0 💤 ±0  0 ❌ ±0 

Results for commit 89ed3c9. ± Comparison against base commit 987dfcb.

♻️ This comment has been updated with latest results.

@andrewlock andrewlock changed the title Interceptors! Add optional ToString() interceptor Oct 12, 2024
@andrewlock andrewlock marked this pull request as ready for review October 12, 2024 18:34
@andrewlock andrewlock merged commit 7041546 into main Oct 13, 2024
5 checks passed
@andrewlock andrewlock deleted the interceptors branch October 13, 2024 09:08
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