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

Register-DbatoolsConfig - Add example that shows -PassThru #8758

Merged
merged 2 commits into from
Feb 15, 2023

Conversation

jpomfret
Copy link
Collaborator

Please read -- recent changes to our repo

On November 10, 2022, we removed some bloat from our repository (for the second and final time). This change requires that all contributors reclone or refork their repo.

PRs from repos that have not been recently reforked or recloned will be closed and @potatoqualitee will cherry-pick your commits and open a new PR with your changes.

  • Please confirm you have the smaller repo (85MB .git directory vs 275MB or 110MB or 185MB .git directory)

Type of Change

  • Bug fix (non-breaking change, fixes # )
  • 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/dataplat/appveyor-lab ?
  • Unit test is included
  • Documentation - Register-DbatoolsConfig dont register in Registry #8368
  • Build system

Purpose

Add an example to show the -PassThru parameter of Set-Dbatoolsconfig

.EXAMPLE
PS C:\> Set-DbatoolsConfig -FullName sql.connection.trustcert -Value $true -PassThru | Register-DbatoolsConfig

Set the `sql.connection.trustcert` configuration to be $true, and then use the -PassThru parameter
Copy link
Member

Choose a reason for hiding this comment

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

The back ticks don't render code statement on our docs page, that is only for markdown support 😉

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Gotcha - shall I remove? Or ok just to show separation?

Copy link
Member

Choose a reason for hiding this comment

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

I'd remove and put in double-quotes or brackets if you need to call focus to it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Consider it done :)

@wsmelton
Copy link
Member

Will go ahead and merge this as the test failures don't pertain to doc changes.

@wsmelton wsmelton merged commit add75a9 into dataplat:development Feb 15, 2023
@jpomfret jpomfret deleted the issue8368 branch February 15, 2023 09:22
@jpomfret
Copy link
Collaborator Author

Thanks @wsmelton!

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.

2 participants