-
Notifications
You must be signed in to change notification settings - Fork 157
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added Update-VSTeamGitRepositoryDefaultBranch #475
Added Update-VSTeamGitRepositoryDefaultBranch #475
Conversation
… the default branch of a repository. - Fixed bad link in README.md
…ze it as something when mocked) Removed call to dot source common.ps1 Removed debug messages Cleaned up comments Bumped Version Updated Changelog
fix: Remove space in -Build paramter of example 4 (MethodsAndPractices#472)
@SebastianSchuetze do I need to re-submit with SupportsShouldProcess implemented? |
What do you mean resubmit? You can just keep editing your branch until everything is fine. You didn't need to close the PR and reopen one. Just commit here until everything is fine. At the end we do a squash merge anyways. |
Thanks, I learn something new every day. I'll try to get this updated on Monday. |
- Added logic to pull refs and validate branch name - Updated call to use splatting for readability
- Added test for non-existent branch name
I believe everything that needed to be fixed has been fixed, including adding support for SupportsShouldProcess / -WhatIf |
I removed some lines in the changelog because it is not interesting if there are typo changes. Only real changes for somebody using the modules is interesting. E.g. bug fixes or new features in cmdlets or the documentation of the cmdlets themselve. approved and merged. Thanks very much!! |
PR Summary
PR Checklist