Skip to content

Conversation

@aniruddhc1
Copy link
Contributor

@aniruddhc1 aniruddhc1 commented Jul 16, 2025

Renames the temporal scaler files to follow the convention used by other scalers in the project.

Renames the temporal scaler files to follow the convention used by other scalers in the project.

Checklist

Fixes #

Relates to #

@aniruddhc1 aniruddhc1 force-pushed the rename-temporal-scaler branch 4 times, most recently from 4c1e94e to 72bf857 Compare July 17, 2025 03:13
Renames the temporal scaler files to follow the  convention used by other scalers in the project.

-  ->
-  ->
-  ->

Signed-off-by: Aniruddh Chaturvedi <[email protected]>
@aniruddhc1 aniruddhc1 force-pushed the rename-temporal-scaler branch from 72bf857 to ab530c4 Compare July 17, 2025 04:05
@rickbrouwer
Copy link
Member

Thanks! This PR is related to discussion here: kedacore/keda-docs#1574

@rickbrouwer
Copy link
Member

rickbrouwer commented Jul 17, 2025

Hi @aniruddhc1 ,

Could you revert this file rename (tests/scalers/temporal_scaler/temporal_scaler_test.go)?
Because this one is in line with the rest.

@aniruddhc1 aniruddhc1 force-pushed the rename-temporal-scaler branch 2 times, most recently from 47aec46 to 15dc6ef Compare July 24, 2025 14:18
Signed-off-by: Aniruddh Chaturvedi <[email protected]>
Copy link
Member

@rickbrouwer rickbrouwer left a comment

Choose a reason for hiding this comment

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

Thanks!

@aniruddhc1
Copy link
Contributor Author

Looks like I need one more reviewer to merge this PR. @rickbrouwer , could you find me someone?

@rickbrouwer
Copy link
Member

No worries, someone will pick it up soon :)

@JorTurFer JorTurFer enabled auto-merge (squash) July 27, 2025 21:16
@JorTurFer
Copy link
Member

/skip-e2e

@JorTurFer JorTurFer disabled auto-merge July 27, 2025 21:19
@JorTurFer JorTurFer merged commit 8a999fc into kedacore:main Jul 27, 2025
24 checks passed
kmoonwright pushed a commit to kmoonwright/keda that referenced this pull request Jul 29, 2025
* refactor: rename temporal scaler files for consistency

Renames the temporal scaler files to follow the  convention used by other scalers in the project.

-  ->
-  ->
-  ->

Signed-off-by: Aniruddh Chaturvedi <[email protected]>

* address feedback

Signed-off-by: Aniruddh Chaturvedi <[email protected]>

---------

Signed-off-by: Aniruddh Chaturvedi <[email protected]>
Signed-off-by: Aniruddh Chaturvedi <[email protected]>
Signed-off-by: kmoonwright <[email protected]>
dpochopsky pushed a commit to dpochopsky/keda that referenced this pull request Sep 12, 2025
* refactor: rename temporal scaler files for consistency

Renames the temporal scaler files to follow the  convention used by other scalers in the project.

-  ->
-  ->
-  ->

Signed-off-by: Aniruddh Chaturvedi <[email protected]>

* address feedback

Signed-off-by: Aniruddh Chaturvedi <[email protected]>

---------

Signed-off-by: Aniruddh Chaturvedi <[email protected]>
Signed-off-by: Aniruddh Chaturvedi <[email protected]>
Signed-off-by: David Pochopsky <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants