Skip to content
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

Fix typo in parameter ReplaceInName #7423

Merged
merged 4 commits into from
Jun 9, 2021
Merged

Fix typo in parameter ReplaceInName #7423

merged 4 commits into from
Jun 9, 2021

Conversation

olegstrutinskii
Copy link
Contributor

Small typo bugfix

Type of Change

  • Bug fix (non-breaking change, no need to create issue )
  • New feature (non-breaking change, adds functionality, fixes # )
  • Breaking change (effects multiple commands or functionality, fixes # )
  • Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
  • Adding code coverage to existing functionality
  • Pester test is included
  • If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
  • Nunit test is included
  • Documentation
  • Build system

The Fixies on their duty.

Approach

The function's parameter will work as designed

Commands to test

Backup-DbaDatabase -SqlInstance SRV-DBSERVER `
    -Path \\SRV-BACKUP\SQLBackup\servername\dbname\backuptype `
    -FilePath dbname_backuptype_timestamp.bak `
    -TimeStampFormat yyyyMMdd_HHmmss `
    -Checksum -Verify -BuildPath -ReplaceInName

@olegstrutinskii olegstrutinskii changed the title Fix typo in parameter ReplaceInName name Fix typo in parameter ReplaceInName Jun 8, 2021
@olegstrutinskii olegstrutinskii marked this pull request as ready for review June 8, 2021 06:14
@nvarscar
Copy link
Contributor

nvarscar commented Jun 8, 2021

There is a surprising number of backup-related failures in Appveyor. .Refresh() might be needed after all

@potatoqualitee
Copy link
Member

Thank you all! 🙇🏼

@potatoqualitee potatoqualitee merged commit 2982345 into dataplat:development Jun 9, 2021
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.

None yet

4 participants