Skip to content

Suppress missing Ruby version errors from extension telemetry#3996

Merged
alexcrocha merged 1 commit intomainfrom
03-06-suppress_missing_ruby_version_errors_from_extension_telemetry
Mar 9, 2026
Merged

Suppress missing Ruby version errors from extension telemetry#3996
alexcrocha merged 1 commit intomainfrom
03-06-suppress_missing_ruby_version_errors_from_extension_telemetry

Conversation

@alexcrocha
Copy link
Contributor

@alexcrocha alexcrocha commented Mar 6, 2026

Introduce NonReportableError base class that MissingRubyError and UntrustedWorkspaceError both extend. The telemetry check in ruby.ts now uses a single instanceof NonReportableError guard, making it extensible without modifying the check for each new error type.

Copy link
Contributor Author

alexcrocha commented Mar 6, 2026


How to use the Graphite Merge Queue

Add the label graphite-merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@alexcrocha alexcrocha force-pushed the 03-06-suppress_missing_ruby_version_errors_from_extension_telemetry branch from 956f2ee to facc782 Compare March 6, 2026 22:44
@alexcrocha alexcrocha added the chore Chore task label Mar 6, 2026
@alexcrocha alexcrocha self-assigned this Mar 6, 2026
@alexcrocha alexcrocha marked this pull request as ready for review March 6, 2026 23:14
@alexcrocha alexcrocha requested a review from a team as a code owner March 6, 2026 23:14
@alexcrocha alexcrocha changed the base branch from 03-06-filter_gemnotfound_and_giterror_setup_errors_from_telemetry to main March 9, 2026 16:19
@alexcrocha alexcrocha force-pushed the 03-06-suppress_missing_ruby_version_errors_from_extension_telemetry branch from facc782 to 52dbb37 Compare March 9, 2026 20:46
Introduce NonReportableError base class that MissingRubyError and
UntrustedWorkspaceError both extend. The telemetry check in ruby.ts
now uses a single instanceof NonReportableError guard, making it
extensible without modifying the check for each new error type.
@alexcrocha alexcrocha force-pushed the 03-06-suppress_missing_ruby_version_errors_from_extension_telemetry branch from 52dbb37 to 058de80 Compare March 9, 2026 20:48
@alexcrocha alexcrocha merged commit c188dc3 into main Mar 9, 2026
36 checks passed
@alexcrocha alexcrocha deleted the 03-06-suppress_missing_ruby_version_errors_from_extension_telemetry branch March 9, 2026 21:28
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