Skip to content

Conversation

@joshuaobrien
Copy link
Contributor

This PR adds a command that opens all of the files touched by the active PR.

Closes #3047

Copy link
Contributor Author

Choose a reason for hiding this comment

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

When we're opening all of the files, we likely don't want to open them as a preview. In order to support this, callers can now specify whether or not files should open as a preview.

Copy link
Member

@alexr00 alexr00 left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, it's looking good! I pushed a change with a few nits changed and left some questions/suggestions.

@alexr00 alexr00 added this to the January 2022 milestone Jan 11, 2022
@alexr00 alexr00 self-assigned this Jan 11, 2022
@joshuaobrien joshuaobrien force-pushed the openAllChangesLocally branch from 2acd664 to c3aa446 Compare January 14, 2022 13:23
@joshuaobrien joshuaobrien requested a review from alexr00 January 14, 2022 13:30
@joshuaobrien joshuaobrien changed the title Add command to open all changes locally Add command to open all diffs Jan 14, 2022
Copy link
Member

@alexr00 alexr00 left a comment

Choose a reason for hiding this comment

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

Looks great, thank you!

@alexr00 alexr00 merged commit 9670562 into microsoft:main Jan 14, 2022
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.

Feature Request: View All Changes In PR

2 participants