Skip to content

Reset the size param between split-pane subcommands#8753

Merged
1 commit merged intomainfrom
dev/migrie/b/split-pane--size-test
Jan 12, 2021
Merged

Reset the size param between split-pane subcommands#8753
1 commit merged intomainfrom
dev/migrie/b/split-pane--size-test

Conversation

@zadjii-msft
Copy link
Member

Summary of the Pull Request

I forgot to reset the --size argument to split-pane when I added it. This PR fixes that, and adds a test so I don't regress it again.

References

PR Checklist

  • I work here
  • Tests added/passed

@zadjii-msft zadjii-msft added Issue-Bug It either shouldn't be doing this or needs an investigation. Product-Terminal The new Windows Terminal. Severity-Blocking We won't ship a release like this! No-siree. Area-Commandline wt.exe's commandline arguments labels Jan 12, 2021
@DHowett DHowett added the AutoMerge Marked for automatic merge by the bot when requirements are met label Jan 12, 2021
@ghost
Copy link

ghost commented Jan 12, 2021

Hello @DHowett!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 8 hours, a condition that will be fulfilled in about 1 hour 40 minutes. No worries though, I will be back when the time is right! 😉

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@DHowett DHowett added the Needs-Second It's a PR that needs another sign-off label Jan 12, 2021
@ghost ghost merged commit cb2cd7e into main Jan 12, 2021
@ghost ghost deleted the dev/migrie/b/split-pane--size-test branch January 12, 2021 23:20
mpela81 pushed a commit to mpela81/terminal that referenced this pull request Jan 28, 2021
## Summary of the Pull Request

I forgot to reset the `--size` argument to `split-pane` when I added it. This PR fixes that, and adds a test so I don't regress it again.

## References
* Missed in microsoft#8543


## PR Checklist
* [x] I work here
* [x] Tests added/passed
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Commandline wt.exe's commandline arguments AutoMerge Marked for automatic merge by the bot when requirements are met Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Second It's a PR that needs another sign-off Product-Terminal The new Windows Terminal. Severity-Blocking We won't ship a release like this! No-siree.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants