Skip to content

Misc typos#6909

Merged
zroubalik merged 20 commits intokedacore:mainfrom
pieterocp:misc-typos
Jul 21, 2025
Merged

Misc typos#6909
zroubalik merged 20 commits intokedacore:mainfrom
pieterocp:misc-typos

Conversation

@pieterocp
Copy link
Contributor

@pieterocp pieterocp commented Jul 15, 2025

Just a bunch of nitpicky typo changes when I was having a look at the pkg/scalers/postgresql_scaler.go to understand how it works under the bonnet, saw a typo in a comment and though if there's one, there might be a few more.

Checklist

  • I have verified that my change is according to the deprecations & breaking changes policy
    • As far as I can tell, this shouldn't lead to visible changes. BUT, this is a bit of a tricky one, since end users could be dependent on the specific writing inside error message, but if they are, then well, happy to pull this out and add in deprecation calls inside the error triggering? 🤔 is required.
  • Changelog has been updated and is aligned with our changelog requirements
  • Commits are signed with Developer Certificate of Origin (DCO - learn more)

Doesn't fix any issues.

(apologies for the mess of force pushes, these should now be signed accordingly)

pieterocp added 19 commits July 15, 2025 17:26
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
…o treat this as immutable)

Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Copy link
Contributor

@dttung2905 dttung2905 left a comment

Choose a reason for hiding this comment

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

Thanks for working on this. Just one nit and we are good to go

Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
@pieterocp pieterocp requested a review from dttung2905 July 16, 2025 08:14
Copy link
Member

@wozniakjan wozniakjan left a comment

Choose a reason for hiding this comment

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

lgtm, thank you!

@wozniakjan
Copy link
Member

wozniakjan commented Jul 16, 2025

/run-e2e
Update: You can check the progress here

Copy link
Member

@zroubalik zroubalik left a comment

Choose a reason for hiding this comment

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

Thank you!

@zroubalik zroubalik merged commit cf579d4 into kedacore:main Jul 21, 2025
27 checks passed
dpochopsky pushed a commit to dpochopsky/keda that referenced this pull request Sep 12, 2025
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
Signed-off-by: David Pochopsky <david.pochopsky@united.com>
tangobango5 pushed a commit to tangobango5/keda that referenced this pull request Dec 22, 2025
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
tangobango5 pushed a commit to tangobango5/keda that referenced this pull request Feb 13, 2026
Signed-off-by: Pieter Oliver <pieter.oliver@nourishcare.com>
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.

4 participants