Skip to content

Conversation

@trevorm4
Copy link
Contributor

@trevorm4 trevorm4 commented May 15, 2025

Updating the README to more accurately reflect the current file structure and fix the various markdown lints


Stack created with Sapling. Best reviewed with ReviewStack.

Updating the README to more accurately reflect the current file structure and fix the various markdown lints
@facebook-github-bot
Copy link
Contributor

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. (Because this pull request was imported automatically, there will not be any future comments.)

@trevorm4 trevorm4 marked this pull request as ready for review May 15, 2025 07:07
Copy link
Contributor

@quark-zju quark-zju left a comment

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.

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in d269d58.

facebook-github-bot pushed a commit that referenced this pull request May 16, 2025
Summary:
Currently ReviewStack doesn't show the number of lines modified like github, so this diff adds a simple display of it. Requires adding 2 additional fields to the graphQL query

Before:
![image](https://github.com/user-attachments/assets/95729ffc-2796-4bf1-b4cc-4ac08f91e8b4)

After:
![image](https://github.com/user-attachments/assets/aaa4cb4d-69d7-4a80-b68c-6dc82b0ff433)

 ---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed with [ReviewStack](https://reviewstack.dev/facebook/sapling/pull/1081).
* __->__ #1081
* #1080

Pull Request resolved: #1081

Reviewed By: quark-zju

Differential Revision: D74788766

fbshipit-source-id: a14aa55821b12f28d8afd9028019d0c58980f540
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.

3 participants