Skip to content

Show reveal_type output under the basic preset' - #5573

Closed
MarcoGorelli wants to merge 1 commit into
meta:mainfrom
MarcoGorelli:basic-reveal-type
Closed

MarcoGorelli wants to merge 1 commit into
meta:mainfrom
MarcoGorelli:basic-reveal-type

Conversation

@MarcoGorelli

@MarcoGorelli MarcoGorelli commented Nov 7, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Fixes #5571. The issue's not been discussed, but this was very simple, and sometimes just submitting a PR is the fastest way to drive a discussion forwards

  • I am a human, and I fully understand the code I am submitting.

Test Plan

@meta-codesync

meta-codesync Bot commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

This pull request has been imported. If you are a Meta employee, you can view this in D130433047. (Because this pull request was imported automatically, there will not be any future comments.)

@MarcoGorelli
MarcoGorelli marked this pull request as ready for review November 7, 2026 15:58
@github-actions

github-actions Bot commented Nov 7, 2026 •

Copy link
Copy Markdown

✅ This PR meets all requirements from our pull request checklist.

Note: this comment updates automatically after changes to the PR. Updates may take up to half an hour.

@stroxler stroxler left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review automatically exported from Phabricator review in Meta.

@meta-codesync

meta-codesync Bot commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

This pull request has been merged in 128c072.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Should --preset=basic show reveal_type?

2 participants