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

Grant-DbaAgPermission - Update Type description #9201

Merged
merged 2 commits into from
Jan 24, 2024
Merged

Grant-DbaAgPermission - Update Type description #9201

merged 2 commits into from
Jan 24, 2024

Conversation

0x7FFFFFFFFFFFFFFF
Copy link
Contributor

The original description is a little bit confusing: Endpoint will modify the DatabaseMirror endpoint type.

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 (affects 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
  • Build system

Purpose

Approach

Commands to test

Screenshots

Learning

The original description is a little bit confusing:
Endpoint will modify the DatabaseMirror endpoint type.
@wsmelton wsmelton changed the title Update Grant-DbaAgPermission.ps1 Grant-DbaAgPermission - Update Type description Jan 7, 2024
Thanks for the suggestion. To me, the original phrase, 'The Endpoint will modify the DatabaseMirror endpoint type,' sounds like some DatabaseMirror endpoint type is being modified. It's in fact the DatabaseMirror endpoint type endpoint that is being modified. But I'm not a native English speaker😂

Co-authored-by: Shawn Melton <[email protected]>
@wsmelton wsmelton changed the base branch from master to development January 24, 2024 17:53
@wsmelton
Copy link
Member

@0x7FFFFFFFFFFFFFFF not sure if your client is doing this but our default branch on the project is development, just for awareness. GitHub UI should be setting the PR target branch to development.

@wsmelton wsmelton merged commit e09b82b into dataplat:development Jan 24, 2024
2 of 3 checks passed
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