Skip to content

Add command to open all diffs - #3238

Merged
Alex Ross (alexr00) merged 10 commits into
microsoft:mainfrom
joshuaobrien:openAllChangesLocally
Jan 14, 2022
Merged

Alex Ross (alexr00) merged 10 commits into
microsoft:mainfrom
joshuaobrien:openAllChangesLocally

Conversation

@joshuaobrien

Copy link
Copy Markdown
Contributor

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

Closes #3047

Comment thread src/view/treeNodes/fileChangeNode.ts Outdated

Copy link
Copy Markdown
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.

@alexr00 Alex Ross (alexr00) left a comment

Copy link
Copy Markdown
Member

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.

Comment thread src/view/treeNodes/fileChangeNode.ts Outdated
Comment thread src/commands.ts Outdated
Comment thread package.json Outdated
@alexr00 Alex Ross (alexr00) added this to the January 2022 milestone Jan 11, 2022
@alexr00 Alex Ross (alexr00) self-assigned this Jan 11, 2022
@joshuaobrien Joshua O'Brien (joshuaobrien) changed the title Add command to open all changes locally Add command to open all diffs Jan 14, 2022

@alexr00 Alex Ross (alexr00) left a comment

Copy link
Copy Markdown
Member

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
Alex Ross (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