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

Update comments to reflect correct name of component #909

Conversation

alexperez52
Copy link
Contributor

Description

Updated comments in 2 files to represent the correct name of component. Load balancer -> Target Allocator
This closes #903

@alexperez52 alexperez52 requested a review from a team June 3, 2022 16:53
@pavolloffay pavolloffay reopened this Jul 11, 2022
@alexperez52
Copy link
Contributor Author

Hi @pavolloffay ! Could this be merged ?

@pavolloffay
Copy link
Member

@alexperez52 CI failed and there is no way to restart it - at least I don't see that option in GHA action.

could you please rebase the PR? It will trigger the CI again.

@frzifus
Copy link
Member

frzifus commented Oct 24, 2022

I assume clicking on update branch should be enough. @alexperez52 or @pavolloffay

@jaronoff97
Copy link
Contributor

We've moved some files around but this is still a problem, I'll open up a fix

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.

Update documentation: Update names Load balancer to Target Allocator
4 participants