Skip to content

Conversation

dbaduck
Copy link

@dbaduck dbaduck commented Aug 20, 2025

There is a misspelling in the JoinAvailablityGroup. Notice the missing i? JoinAvailabilityGroup is what it should be. This is impacting dbatools module as well as others that use the JoinAvailabilityGroup method on AvailabilityDatabase.

There is a misspelling in the JoinAvailablityGroup. Notice the missing i? JoinAvailabilityGroup is what it should be. This is impacting dbatools module as well as others that use the JoinAvailabilityGroup method on AvailabilityDatabase.
@shueybubbles
Copy link
Collaborator

thx for the PR!
Since this is a public method, changing the name is breaking, so create a new method with the proper name and mark the old one with [Obsolete]. Have the old one call the new one.

@dbaduck
Copy link
Author

dbaduck commented Sep 11, 2025

It is misspelled, I need it changed back as it used to work. I would call this a regression bug, not a new method. Has it really been this spelling all along?

@shueybubbles
Copy link
Collaborator

It is misspelled, I need it changed back as it used to work. I would call this a regression bug, not a new method. Has it really been this spelling all along?

yes it's been this way since before 2016 according to the source history internally

@dbaduck
Copy link
Author

dbaduck commented Sep 11, 2025

That is amazing. I will file a bug on the other side (consumer) then and get it to match.
Any plans to fix the spelling?

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